Miscellaneous R2RDump improvements (#23215)
authorTomáš Rylek <trylek@microsoft.com>
Mon, 25 Mar 2019 16:22:46 +0000 (17:22 +0100)
committerJan Kotas <jkotas@microsoft.com>
Mon, 25 Mar 2019 16:22:46 +0000 (09:22 -0700)
commit368639e46a903b6ff44a21d67b0e0fea661c91d7
tree0fa4e61c44710e78cfe22e0579a0e428e29c0032
parentb6b3d15118c47380ced62f99767f25fac44fad4c
Miscellaneous R2RDump improvements (#23215)

1) In Naked mode, hide the distinction amongst METHOD_ENTRY vs.
METHOD_ENTRY_REF_TOKEN vs. METHOD_ENTRY_DEF_TOKEN as it's not
important for correctness and causes undesirable churn in diffs.
This required propagating the DumpOptions around in a couple
of places.

2) For historical reasons, the DisassemblingTypeProvider had
its own implementations of metadata formatting that is now provided
by MetadataNameFormatter. I have removed at least a part of this
duplication in this change. [It was causing undesirable diffs as
one version used to output type owner assemblies whereas the other
did not.]

Thanks

Tomas
src/tools/r2rdump/DisassemblingTypeProvider.cs
src/tools/r2rdump/R2RDump.cs
src/tools/r2rdump/R2RReader.cs
src/tools/r2rdump/R2RSignature.cs