This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 4/6] command 'set debug notification'.


> This patch is V2, and in V3, zuinterger is used.  Looks some debug
> commands are using boolean and some are using zuinteger.  I agree
> that we can use boolean here.  I'll update this patch in V3 series.

FWIW: I used to prefer boolean, because you can just "set debug blah",
and it'll enable the debug traces. But some modules have so many
things to trace about that several levels of verbosity are required.
In that case, zuinteger becomes necessary.

Should the needs change over time, I don't think it would be a big deal
to change the type of the setting from one type (boolean) to another
(zuinteger), since this is a debug setting.  But, technically, that's
still a non-upward-compatible change - unless we add some extra glue,
I suppose.

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]