Hmmm.... If you want to rewrite a method from scratch, there's a whole lot of other stuff you need to remove. I'm doing the following in a project of mine:

method.Body.Instructions.Clear();
method.Body.Variables.Clear();
method.Body.ExceptionHandlers.Clear();

method.DebugInformation.SequencePoints.Clear();
method.DebugInformation.Scope = null;
method.DebugInformation.StateMachineKickOffMethod = null;

method.CustomDebugInformations.Clear();

On the other hand, the ILProcessor only deals with the instruction list today, so I suppose making it clear a bunch of other stuff would not be appropriate.

Hmmm.... If you want to rewrite a method from scratch, there's a whole lot of other stuff you need to remove. I'm doing [the following in a project of mine](https://github.com/ltrzesniewski/InlineIL.Fody/blob/2b63811da7cf22473a174989f98f225a3083c136/src/InlineIL.Fody/ModuleWeaver.cs#L63-L71): ```C# method.Body.Instructions.Clear(); method.Body.Variables.Clear(); method.Body.ExceptionHandlers.Clear(); method.DebugInformation.SequencePoints.Clear(); method.DebugInformation.Scope = null; method.DebugInformation.StateMachineKickOffMethod = null; method.CustomDebugInformations.Clear(); ``` On the other hand, the `ILProcessor` only deals with the instruction list today, so I suppose making it clear a bunch of other stuff would not be appropriate.

jbevain commented

2020-05-21 12:22:43 -05:00

(Migrated from github.com)