This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PING] [RFC] Extend gdb_ari.sh with ARI_OFF/ARI_ON
- From: Tom Tromey <tromey at redhat dot com>
- To: "Pierre Muller" <pierre dot muller at ics-cnrs dot unistra dot fr>
- Cc: <gdb-patches at sourceware dot org>
- Date: Fri, 21 May 2010 10:17:26 -0600
- Subject: Re: [PING] [RFC] Extend gdb_ari.sh with ARI_OFF/ARI_ON
- References: <10524.3587088355$1274389681@news.gmane.org>
- Reply-to: tromey at redhat dot com
>>>>> "Pierre" == Pierre Muller <pierre.muller@ics-cnrs.unistra.fr> writes:
Pierre> Noone reacted to that proposal yet...
Sorry about that.
Pierre> We this new patch, we can disable all 8 warnings of windows-nat.c
Pierre> by adding just two lines:
Pierre> /* ARI_OFF: editCase function */
Pierre> and
Pierre> /* ARI_ON: editCase function */
Pierre> at the start and the end of the list of
Pierre> bad_MixedFunctionNameSameAsWindowsAPI
Pierre> list.
Pierre> Comments?
I think this kind of mechanism is too obscure. The markers may be very
far from the place that one is editing.
Do we really need this particular ARI rule at all? This sort of problem
is usually caught by patch review... and if it an instance is missed, it
doesn't have any impact on code quality.
Tom