Hello, I'm out of this conversation but, Is it going to work with GDB 6.8.5?<br><br><div class="gmail_quote">On Sat, Jun 13, 2009 at 4:28 PM, Niko Sams <span dir="ltr"><<a href="mailto:niko.sams@gmail.com">niko.sams@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5">On Sat, Jun 13, 2009 at 19:47, Vladimir Prus<<a href="mailto:ghost@cs.msu.su">ghost@cs.msu.su</a>> wrote:<br>


> On Saturday 13 June 2009 Niko Sams wrote:<br>
><br>
>> On Sat, Jun 13, 2009 at 14:51, Vladimir Prus<<a href="mailto:ghost@cs.msu.su">ghost@cs.msu.su</a>> wrote:<br>
>> > On Saturday 13 June 2009 Niko Sams wrote:<br>
>> ><br>
>> >> Resending mail because the attachment is too large.<br>
>> >><br>
>> >> uploaded the patch here:<br>
>> >> <a href="http://www.vivid-planet.com/upload/dbg.diff" target="_blank">www.vivid-planet.com/upload/dbg.diff</a><br>
>> ><br>
>> > As I've said on IRC, I have two concerns about this patch:<br>
>> ><br>
>> > 1. It's big.<br>
>> True. It's not just a patch to fix locals - it's a replacement for the whole<br>
>> locals/watches/tooltip implementation. (locals are just the first<br>
>> thing I got working)<br>
>> And as the code is taken from an well working application (they even<br>
>> released a stable version)<br>
>> I think it should be not that bad.<br>
><br>
> It's probably not that bad, but I would not call it strictly better<br>
> than either KDevelop 3 or KDevelop 4.<br>
><br>
>> > 2. Substantial parts of it deal with debug helpers for display of<br>
>> > STL and other things, and we'll get that mostly for free in GDB 7.0.<br>
>> That debug helper stuff in my patch is all commented out. I'd love to drop<br>
>> that if we can use gdb python scripts for that.<br>
>><br>
>> > I've tried to hack something myself, and the patch is attached. While<br>
>> > it presently crashes due to me messing up order of destruction somewhere,<br>
>> > I think it will give us >50% of what we want from local variables display,<br>
>> > and it's much more smaller. As for remaining <50%, it's probably a UI<br>
>> > issue that I'll post about separately.<br>
>> If you want to complete the current code I'm of course fine with that and will<br>
>> drop the qt-creator code.<br>
>><br>
>> However if I will work on that we should consider using qt-creator code:<br>
>> - imho it's much easier to understand (I lost motivation when looking at<br>
>> those Variable* classes - there are so many of them related to each other)<br>
><br>
> I see only 2 (Variable and VariablesRoot) :-)<br>
><br>
>> - it's easier to port something that works<br>
>> - it's maintained by nokia, we can integrate their patches<br>
><br>
> I am not actually convinced that grabbing a bigger codebase<br>
> in order to get local variables is the best idea. And porting<br>
> patches for 500K codebase is a problem in itself.<br>
</div></div>As I said - it's not just about getting local variables.<br>
<div class="im"><br>
>> - we don't need to take everything (debug helper)<br>
>> - there are debugger independent classed we can just put into the platform and<br>
>> let other plugins use it<br>
><br>
> What are those debugger independent classes? If they are worthwhile and independent,<br>
> we certainly can grab them.<br>
</div>I was talking about WatchHandler and WatchData. Those would be instead of<br>
Variable and VariableCollection. So if we use the current code we would have<br>
to make Variable and VariableCollection generic.<br>
<div class="im"><br>
>> We have two possibilities:<br>
>> 1. Use my patch and continue working on it<br>
>> 2. Use the current implementation, refactor debugger independent classes out<br>
><br>
> I though you have already did some kdevplatfrom/kdevelop split. Am I misunderstand<br>
> you?<br>
</div>yes, but not variables. That's the main thing missing.<br>
<div class="im"><br>
>> and make it work.<br>
><br>
> After poking at debugger yesterday and today, I get the impression that "make it work"<br>
> is past, what is needed now is making it work good.<br>
</div>sounds good, but continue that poking :D<br>
<br>
I'll leave my experiment for now and try to get into the current variable code.<br>
<font color="#888888"><br>
<br>
Niko<br>
</font><br>
PS:<br>
Just played around with the qstring pretty printer you sent - works great!<br>
Soo much simpler than this debug helper.<br>
<div><div></div><div class="h5"><br>
_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br>