[PATCH] GDB friendly assert

Andrew Cagney ac131313@cygnus.com
Thu Mar 1 09:06:00 GMT 2001


Andrew Cagney wrote:
> 
> Mark Kettenis wrote:
> >
> >    Date: Tue, 24 Oct 2000 23:46:47 +1100
> >    From: Andrew Cagney <ac131313@cygnus.com>
> >
> >    Thanks!  Good move.
> >
> >    My only (A.R.) thought is to make the macro upper case so it is very
> >    clear to developers that they are  using a MACRO and not a function.
> >
> > Hmm, my idea was that it simply is the standard assert() (which
> > typically is a macro) with gdb_ prepended.
> 
> Yes, I know.  I'm trying to trade off ``MACRO's are upper case'' against
> ``assert()'' in lower-case is more consistent with STDC even if it was a
> bad idea.
> 
> >  But if you feel it should
> > be changed into GDB_ASSERT, I'll be happy to change it.
> 
> I don't know :-)

To close this, I'm going to add a comment to "gdb_assert.h" noting that
there is a tradeoff between these two.  lcase was chosen as it is closer
to the existing UNIX convention.

As a btw, people adding new code may want to use gdb_assert() in
preference to internal error.  It is shorter and more convenient.

	Andrew



More information about the Gdb-patches mailing list