winedbg: resend - add display for floating point status

Eric Pouech eric.pouech at wanadoo.fr
Mon Mar 20 14:07:51 CST 2006


I'm getting picky, but there are a couple of things you still need to 
work on:
- first, I don't think it's worthwhile to add another method to the CPU 
backend. Adding a boolean (as parameter) to print_context should do.
- you also need to fix all the be_*.c files (for amd, ppc...) as well. 
Otherwise, they won't compile. The rule of thumb is never commit a patch 
that breaks compilation on some platforms.
- one of the test looks suspicious (shouldn't it be if 
(!(ctx->loatSave.StatusWord & 0x00000001)) instead ?)
> +       if (!ctx->FloatSave.StatusWord & 0x00000001)
A+

-- 
Eric Pouech




More information about the wine-devel mailing list