On 25.02.2013, at 14:02, Andreas Färber wrote: > Am 25.02.2013 13:49, schrieb Alexander Graf: >> >> On 21.02.2013, at 05:24, Andreas Färber wrote: >> >>> There's an opcode handler field dependent on PPC_DUMP_CPU without which >>> the build fails. >>> >>> Signed-off-by: Andreas Färber <afaer...@suse.de> >>> --- >>> target-ppc/translate.c | 1 + >>> target-ppc/translate_init.c | 1 - >>> 2 Dateien geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) >>> >>> diff --git a/target-ppc/translate.c b/target-ppc/translate.c >>> index 2ac5794..2e74e45 100644 >>> --- a/target-ppc/translate.c >>> +++ b/target-ppc/translate.c >>> @@ -33,6 +33,7 @@ >>> >>> /* Include definitions for instructions classes and implementations flags */ >>> //#define PPC_DEBUG_DISAS >>> +#undef PPC_DUMP_CPU >> >> #undef? > > // is not permitted. :) > Alternative would be /* #define ... */ > > Just edit the line to your liking. :)
The current coding style for debug defines is // #define DEBUG_FOO. It don't think it makes sense to deviate from that notion unless we do it consistently. And to do that, we need to consistentify the handling first which your patches do. So IMHO I'd rather like to see a patch changing the style to whatever people prefer after this set is through. Alex
