This is the mail archive of the gdb-patches@sources.redhat.com 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 to testsuite/gdb.c++/namespace.exp


Michael Snyder wrote:
> Would it not be better, then, if the test accepted both 
>  \0 and \000?

I thought about that.  It would be better; and if I had written the
patch, I would have written it that way.

But there's so much to do in C++ land, and so little time, that I'd
rather let it be.  I'm 100x more worried about things like this:

  # native i686-pc-linux-gnu
  # gdb HEAD (2001-12-22)
  # gcc 2.95.3
  # -gdwarf-2
  print this^M
  $1 = (A * const) 0xa^M
  (gdb) PASS: gdb.c++/method.exp: print this (in foo)

gdb is printing an incorrect value and the testsuite does not catch it.
This is a bug in gdb and a shortfall in the testsuite and it affects
real users.

I am organizing my life so that I have more time to care about bugs
like that.  That means conserving attention on lesser things, to their
detriment.  It's an attention-triage question.

But if this issue bothers you enough to submit a patch, I will approve it.

Michael C


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