|
From: Dirk M. <dm...@gm...> - 2005-07-23 00:36:23
Attachments:
vd.diff
|
Hi, Rich Coe <Ric...@me...> wants me to apply the following patch to 2.4.x tree, it doesn't seem to be in 3.0 either. I currently don't have time to review it, does it look sane? error is: ==4089== Memcheck, a memory error detector for x86-linux. ==4089== Copyright (C) 2002-2005, and GNU GPL'd, by Julian Seward et al. ==4089== Using valgrind-2.4.0, a program supervision framework for x86-linux. ==4089== Copyright (C) 2000-2005, and GNU GPL'd, by Julian Seward et al. ==4089== ==4089== My PID = 4089, parent PID = 3566. Prog and args are: ==4089== ...... @@ don't know what type 'p' is @@ parsing pair<char* const, Commentclass::_comment*>,std::allocator<std::pair<char* const, Commentclass::_comment*> >,true>:(903,6),0,256;; gave NULL type (air<char* const, Commentclass::_comment*>,std::allocator<std::pair<char* const, Commentclass::_comment*> >,true>:(903,6),0,256;; remains) @@ expected ':' at struct method MANGLE-ARGS (remains="ir<char* const, Commentclass::_comment*>,std::allocator<std::pair<char* const, Commentclass::_comment*> >,true>:(903,6),0,256;;") @@ parsing (903,9)=s4_Rb_tree_alloc_base<std::pair<char* const, Commentclass::_comment*>,std::allocator<std::pair<char* const, Commentclass::_comment*> >,true>:(903,6),0,256;; gave NULL type (s4_Rb_tree_alloc_base<std::pair<char* const, Commentclass::_comment*>,std::allocator<std::pair<char* const, Commentclass::_comment*> >,true>:(903,6),0,256;; remains) Dirk |
|
From: Tom H. <to...@co...> - 2005-07-23 06:39:02
|
In message <200...@gm...>
Dirk Mueller <dm...@gm...> wrote:
> Rich Coe <Ric...@me...> wants me to apply the following patch to
> 2.4.x tree, it doesn't seem to be in 3.0 either. I currently don't have time
> to review it, does it look sane?
I saw that before but he didn't provide any kind of explanation about
what he that the problem was or how his patch fixed it and as it was
playing with a very fragile area of the code I didn't pursue it any
further - I figured if he really wanted it applied he would open a
ticket.
Bascially anything that plays with the C++ name parsing in stabs needs
to be considered *extremely* carefully and trying to work out whether
this is a good patch from the patch alone with no test case or details
of the problem is way too hard.
Tom
--
Tom Hughes (to...@co...)
http://www.compton.nu/
|
|
From: Julian S. <js...@ac...> - 2005-07-23 08:42:20
|
> On Saturday 23 July 2005 07:38, Tom Hughes wrote: > Bascially anything that plays with the C++ name parsing in stabs needs > to be considered *extremely* carefully and trying to work out whether > this is a good patch from the patch alone with no test case or details > of the problem is way too hard. I agree with Tom. Too risky, particularly given that there is no justification/explaination for it. J |
|
From: Rich C. <Ric...@me...> - 2005-08-02 16:39:16
|
I'll narrow down to which library is causing the problem, and gin up a test case. AFAIK, it's stabs output from g++ that makes V crash. On Sat, 23 Jul 2005 09:43:44 +0100 Julian Seward <js...@ac...> wrote: > > > On Saturday 23 July 2005 07:38, Tom Hughes wrote: > > Bascially anything that plays with the C++ name parsing in stabs needs > > to be considered *extremely* carefully and trying to work out whether > > this is a good patch from the patch alone with no test case or details > > of the problem is way too hard. > > I agree with Tom. Too risky, particularly given that there is no > justification/explaination for it. > > J > > > > ------------------------------------------------------- > SF.Net email is sponsored by: Discover Easy Linux Migration Strategies > from IBM. Find simple to follow Roadmaps, straightforward articles, > informative Webcasts and more! Get everything you need to get up to > speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click > _______________________________________________ > Valgrind-developers mailing list > Val...@li... > https://lists.sourceforge.net/lists/listinfo/valgrind-developers -- Rich Coe ric...@me... General Electric Healthcare Technologies Global Software Platforms, Computer Technology Team |