|
From: George S. <st...@kd...> - 2004-03-04 23:21:00
|
On Thursday 04 March 2004 17:42, Jeremy Fitzhardinge wrote:
> No, that's exactly right. The debug contains a scope which ends before
> it begins. I may need to look into it in more detail (there could be a
> meaningful interpretation of this), but the simple fix, which is in CVS
> HEAD, is this:
>
> coregrind/vg_symtab2.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff -puN coregrind/vg_symtab2.c~fix-negscope coregrind/vg_symtab2.c
> --- valgrind/coregrind/vg_symtab2.c~fix-negscope 2004-02-05
> 14:00:54.115608640 -0800 +++
> valgrind-jeremy/coregrind/vg_symtab2.c 2004-02-05 14:01:15.046426672 -0800
> @@ -301,8 +301,8 @@ void VG_(addScopeInfo) ( SegInfo* si,
> Int size = next - this;
> ScopeRange range;
>
> - /* Ignore zero-sized scopes */
> - if (this == next) {
> + /* Ignore zero-sized or negative scopes */
> + if (size <= 0) {
This is exactly the first thing I tried, and then I got more errors after
that, so I assumed it was more complex. I'm going to try the latest snapshot
of PPC valgrind and if this fix is not included, I'll add it in again.
However the log looked worse last time. :-) I'll post the new results when I
get them.
FYI: I'm using a much different system than Paul - different glibc,
compiler, and probably kernel. I guess we can expect some differences.
--
George Staikos
KDE Developer http://www.kde.org/
Staikos Computing Services Inc. http://www.staikos.net/
|