This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Stabs parsing regression from GDB 6.6 to GDB 6.6.90
- From: Pedro Alves <pedro_alves at portugalmail dot pt>
- To: Joel Brobecker <brobecker at adacore dot com>, Pierre Muller <muller at ics dot u-strasbg dot fr>, 'Pedro Alves' <pedro_alves at portugalmail dot pt>, gdb-patches at sourceware dot org
- Date: Wed, 03 Oct 2007 22:34:17 +0100
- Subject: Re: [RFC] Stabs parsing regression from GDB 6.6 to GDB 6.6.90
- References: <46F486B4.6050900@portugalmail.pt> <46F56F04.6070601@portugalmail.pt> <46F707C3.1090105@portugalmail.pt> <006101c7fe8b$70d5af70$52810e50$@u-strasbg.fr> <4053daab0709240321n40d7e3e0vc0f7d5567e990785@mail.gmail.com> <006701c7feae$fbb75850$f32608f0$@u-strasbg.fr> <46F8C1C8.7080608@portugalmail.pt> <46F9A061.2020909@portugalmail.pt> <001b01c805b5$ce7f0730$6b7d1590$@u-strasbg.fr> <20071003184156.GC4305@adacore.com> <20071003185126.GA1976@caradoc.them.org>
Daniel Jacobowitz wrote:
On Wed, Oct 03, 2007 at 11:41:56AM -0700, Joel Brobecker wrote:
I definitively think that this patch should go both into
head and branch, but it needs approval from someone else...
Unless someone like Elena who knows this code very well says it's safe,
I would prefer if the patch didn't get applied to the branch. I'm a bit
concerned about it given the few iterations that you both had to go
through in order to get it to that point. It just shows that it is
not obvious.
That's because the original code to read signed octals was never
triggered from day 1. I suspect it was due to a bad/incomplete
merge from whatever tree it came from, and a testcase that
doesn't really trigger it.
I think it's a bad idea to ship a release of GDB with a new bug that
we know about, which wasn't present in the previous release.
Another option is to revert this on the branch:
2007-08-04 Michael Snyder
* stabsread.c (read_huge_number): Attempt to compute value before
values that it depends on.
Since nobody complained until then (except for Coverity). The signed
octal parsing was dead code Before Michail's Patch (BMP), and is broken
after it. BMP, the basic types happened to be recorded with the
correct bit sizes, but ADA ranges that have a negative edge (-50, 50)
would be misrecorded . That was what the original code attempted to
fix, but I don't really know where it makes a difference. Perhaps
a print command which sets a value whose type is covered by a
range, with an out-of-range value, will trigger a warning:
print var = -51
warning: -51 is out of range for type x [-50, 50].
If gdb doesn't do it yet, it could be useful anyway.
What I know of ADA is what one can learn from a 10 minutes HOWTO reading.
I read through the patch. It seems fine to me, though I am mostly
trusting Pedro, Pierre, and their test cases.