[PATCH] Span frame names in debugging watch window

Hamish Rodda rodda at kde.org
Sat Dec 6 11:35:05 UTC 2003


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Thu, 04 Dec 2003 05:51 pm, jbb wrote:
> On Thu, 04 Dec 2003 18:48, Hamish Rodda wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > Hi,
> >
> > I got annoyed with the watch window, having to give the first column more
> > space than it really needs just so I could see the frame name, so I wrote
> > a patch to span the first column for the frame name across the remaining
> > columns.  I've attached a screenshot and the patch.
> >
> > It's a bit of a hack, and really should be done within qt.  However I
> > can't forsee a better version of this functionality getting into qt
> > before 4.0. This patch produces an ok result.  It can be cleaned up
> > significantly: I think I can eliminate the lag in redrawing when resizing
> > columns by directly calling the paint function myself, and instead of
> > switching to QObjects I can have an internal QObject just for the items
> > that require it. I'm posting it now to see if I should bother continuing
> > with it (ie. whether it is likely to be accepted or not).
> >
> > Comments?
>
> A very good idea I think :-)
> Now just waiting for the next version...

Ok, here it is.  The QObject stuff is moved into a private QObject so doesn't 
burden the other items.  The redraw is as fixed as it is ever going to get 
without changing qt or fiddling with the parent listview widget - there's 
some barely noticable jittering, but that's it.  Apart from that, it behaves 
well for me, and noticeably improves the usability of the widget :)

So, now it needs some thorough testing...  apply patch in 
languages/cpp/debugger.

Cheers,
Hamish.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (GNU/Linux)

iD8DBQE/0bC5H8BtnSmIlUYRAtAwAJ9vxy9nVzgwiFGMIvOQQPPZYLBN5wCfWw4j
AXnc2MpcchPzPcIyUzI+KOs=
=12vx
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdevelop-20031206.patch
Type: text/x-diff
Size: 9746 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20031206/19b33068/attachment.bin>


More information about the KDevelop-devel mailing list