Re: [Pnet-developers] [patch #3599] ildasm offsets

[Rhys Weatherley]

On Thursday 01 January 1970 10:00 am, James Michael DuPont wrote:
> It should be turned into a command line option
> for ildasm. This is needed information for debugging verifier information.
> Any *user* of ildasm who is debugging verifier issues will want to have
> this information.

Right now, "any user of ildasm" seems to be just you.  I've never had any 
trouble matching up the offsets when these issues have occurred in the past.

It is great that this helps you find bugs, but no one else finds bugs this 
way.  If we added debug dump modes that were handy for everyone's personal 
debugging style, there would be nothing but debug code in the system.

Debug code should generally be removed after the bug has been located.  It 
keeps the system cleaner and easier to understand.  A few modes, like CVM 
debugging, are present because we have to use them so often.   But they are 
the exception, not the rule.

By the way, I'm still waiting on that patch for "sizeof" in the C compiler, 
which you promised to work on that over a month ago.  And I even told you 
where to apply the fix.  It shouldn't have taken you more than a few days.

Cheers,

Rhys.