search

0xd4d / dnlib

Reads and writes .NET assemblies and modules
MIT License
2.14k stars 583 forks source link

Update comment for DynamicMethodBodyReaderOptions #554

Closed CreateAndInject closed 5 months ago

CreateAndInject commented 5 months ago

Close #552

wtfsck commented 5 months ago

This PR does more than the PR title says. Why did you add new code?

wtfsck commented 5 months ago

I see, seems like it's based on your other PR https://github.com/0xd4d/dnlib/pull/553 .

You should create a new branch with just the comment fix and force push.

CreateAndInject commented 5 months ago

I see, seems like it's based on your other PR #553 .

You should create a new branch with just the comment fix and force push.

Maybe because I modify the same line? Maybe File changed will show this commit change only when that PR merged. Let's see how it shows when that PR merged to decide if it need be changed.