[Bug 72830] Crash when switching between compiling output levels

Mantia Andras amantia at freemail.hu
Fri Jan 23 09:12:05 UTC 2004


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
      
http://bugs.kde.org/show_bug.cgi?id=72830      




------- Additional Comments From amantia at freemail.hu  2004-01-23 09:11 -------
Subject: Re:  Crash when switching between compiling output levels

> Hmm.. inefficient, yes.. But it's actually QTextEdit that's blowing up. I 
believe this is the line that does it:
> > popup->setItemEnabled( d->id[ IdSelectAll ], (bool)text().length() );
>
> It's pushing all of the widgets contents into a QString only to call
> length() on it to make sure it isn't zero.. Doesn't seem to be the most
> efficient way of doing it.. :)

Might be. But where is this code? I can't find it. Just because QTextEdit has 
a length method, so there is no need to call text() first. Oh, I see, it's in 
QTextEdit code. And idea would be to not call QTextEdit::createPopupMenu at 
all, just that we need to handle the Copy and Select All case in the 
makewidget in this case. Shouldn't be so hard.

Andras




More information about the KDevelop-devel mailing list