Bug 57364 - grep view doesn
Summary: grep view doesn
Status: RESOLVED FIXED
Alias: None
Product: kdevplatform
Classification: Developer tools
Component: editor integration (show other bugs)
Version: unspecified
Platform: Mandrake RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: KDevelop Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-04-17 10:55 UTC by Frederic Heem
Modified: 2008-08-23 22:54 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frederic Heem 2003-04-17 10:55:12 UTC
Version:           3.0.0a4 (using KDE 3.1.0)
Installed from:    Mandrake Linux Cooker i586 - Cooker
Compiler:          gcc version 3.2.2 (Mandrake Linux 9.1 3.2.2-3mdk)
OS:          Linux (i686) release 2.4.21-0.13mdkenterprise

The grep view is sometimes unreliable, it opens the good file but doesn
Comment 1 Jens Dagerbo 2003-04-20 19:31:41 UTC
Judging by that debug output, the problem is almost certainly not in gideon, but more 
likely in your editor part. What editor part do you use? Kate? 
Comment 2 Joachim Eibl 2003-04-21 18:05:24 UTC
Hi, 
 
I use kdevelop 3.0a4a and have a problem with the grep-function: 
 
When I scroll down in the grep-results, and select a result-line then the 
editor shows a wrong position. This doesn't happen, if I don't scroll down. 
Another precondition is that I have selected "Devices -> Mouse -> 
Double click to open files/directories" in the control-center. When I use 
"Single click to open files/directories" everything works fine. 
 
I've analyzed the problem a bit more and the problem seems to be the 
KListBox-implementation, which doesn't behave correctly for "Double-clickers". 
 
Nevertheless I would recommend to change the GrepViewWidget so that it always 
connects the signal clicked() instead of executed(). 
 
kdevelop-3.0a4a/parts/grepview/grepviewwidget.cpp 
... 
GrepViewWidget::GrepViewWidget(GrepViewPart *part) 
    : ProcessWidget(0, "grep widget") 
    , m_matchCount(0) 
{ 
    grepdlg = new GrepDialog(this, "grep widget"); 
    connect( grepdlg, SIGNAL(searchClicked()), 
             this, SLOT(searchActivated()) ); 
    connect( this, SIGNAL(clicked(QListBoxItem*)),  // clicked instead of executed 
             this, SLOT(slotExecuted(QListBoxItem*)) ); 
 
    m_part = part; 
} 
... 
 
Best regards, 
Joachim 
joachim.eibl@gmx.de 
 
PS: Otherwise I use Suse 8.2, with KDE 3.1.1. 
Comment 3 Harald Fernengel 2003-04-22 01:39:05 UTC
Fixed in CVS, thanks for the patch 
Comment 4 Frederic Heem 2003-05-27 11:27:16 UTC
Sorry, but even when single click is selected, the problem persists. FYI, the latest 
gideon from CVS is used.  
Comment 5 Jens Dagerbo 2003-05-27 18:58:59 UTC
Just to make sure: the file you click isn't open and edited? grep works on disk, so if you 
have made any unsaved changes, the line numbers will not match up. 
 
Just to make sure we're not chasing ghosts... :) 
Comment 6 Frederic Heem 2003-05-28 18:04:08 UTC
Not at all, sometines the grep view opens a file and go the the first line. And 
sometimes it makes the whole gideon crashed and prints this message: 
kdevelop (grep view): context in grepview 
kdevelop (core): Word:iConnectedSocket: 
kdevelop (grep view): context in grepview 
kdevelop (core): Changing to dir 
kdevelop (core): [virtual void PartController::editDocument(const KURL&, int)] 
file:/user/fred/cvs/communitel_3_13/netergy/edev/sw/src/protocols/call/h323/ccapi/src/h323ccapi.c 
linenum 4287 
kdevelop (core): mimeType = text/x-csrc 
kdevelop (core): factory = 0x87a6ba0 
kdevelop (core): Encoding: , extension: 0x9f76948 
kdevelop (core): MainWindowShare::slotGUICreated() 
kdevelop (core):  *** found "set_confdlg" action - unplugging 
qeditor: CppSupportPart::activePartChanged() 
kdevelop (core): dcop emitting loadedFile 
/user/fred/cvs/communitel_3_13/netergy/edev/sw/src/protocols/call/h323/ccapi/src/h323ccapi.c 
Alarm clock 
 
Comment 7 Frederic Heem 2003-05-28 18:14:50 UTC
Changing the editor from Embedded Text Editor Advanced to Qt Designer Text 
editor solves this problem.  
Comment 8 Jens Dagerbo 2003-05-28 19:24:49 UTC
>sometines the grep view opens a file and go the the first line 
 
Yeah, this is a known katepart bug. This can happen for large files (that isn't already 
open). I've seen it with files 2000+ lines.  
 
Never seen that crash though. 
Comment 9 Christoph Cullmann 2003-09-07 22:50:25 UTC
this behaviour is fixed for kde head's kate, it now load the file syncronous like qeditor, 
therefor such set cursor stuff works after openURL returned