[dbghelp] continue dwarf support

Eric Pouech eric.pouech at wanadoo.fr
Thu Oct 13 13:53:27 CDT 2005

Raphael wrote:
>>but it's of no use with the patch you sent, so why clobber the patch
>>with this ?
> no because i think its better to avoid assert in this case on dbghelp code :)
> But if you prefer i can restore original code
IMO the correct fix would in field insertion to return one of three values:
- success (didn't exist before)
- did exist before
- couldn't add
(instead of the current success/failure mode)
so that every debug info parser can pick what it wants
my main objection here is that you're making modifications for something 
you don't use (in the patch you sent). For readability and acceptance of 
patches it's better to only include it if you make use of it.


Eric Pouech

More information about the wine-devel mailing list