Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
RE: [cdt-dev] Detailed formatter feature for CDT, feedback welcome!

I agree, this feature is good addition to CDT. Thanks for the patch.
But the code requires changes. I have looked at it briefly and noticed that ICType is used in ICDITarget. ICType is not a CDI interface and shouldn't be referenced by it.

-----Original Message-----
From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Warren.Paul@xxxxxxxxx
Sent: Friday, June 20, 2008 3:01 PM
To: cdt-dev@xxxxxxxxxxx
Subject: RE: [cdt-dev] Detailed formatter feature for CDT, feedback welcome!

I haven't looked at the code, just the screen shots.  But this is something that's I've been wanting to tackle for some time now but just haven't had the cycles.  I think this would be a very nice feature addition to CDT.  The UI looks good to me.  We need to be careful with the back-end implementation details though because there are many debuggers used in CDT besides GDB.  There probably needs to be an extension point as well to allow pre-population of formatters.


-----Original Message-----
From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of ext Tor Arne Vestbø
Sent: Friday, June 20, 2008 6:48 AM
To: cdt-dev@xxxxxxxxxxx
Subject: [cdt-dev] Detailed formatter feature for CDT, feedback welcome!


I've spent a few of my creative Fridays implementing the JDT Details Formatter feature for CDT, and would like some feedback on my approach.

You can find the branch here:

Right now it's just a big commit, but I could try to divide it into smaller pieces if possible.

My approach was to copy as much of the GUI stuff from JDT, and then re-implement the back-end using GDB macros, which are loaded into the debug target. That way we could perhaps move the common parts to platform later on.

There' still a lot of stuff missing, like showing the result for mouse hover, or having formatters for base classes apply to subclasses too, but the general framework is there.

Here's a few screenshots:

What do you think?

Tor Arne

Tor Arne Vestbø, Software Engineer
Trolltech ASA, Oslo, Norway
cdt-dev mailing list
cdt-dev mailing list

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.

Back to the top