[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kdevelop-devel
Subject:    Code completion deadlock (again) and patch
From:       Jens Zurheide <jens.zurheide () gmx ! de>
Date:       2005-09-21 21:46:20
Message-ID: 200509212346.22833.jens.zurheide () gmx ! de
[Download RAW message or body]

Hi,

I thought it is worth looking into the C++ code completion problems I 
encounter with kdevelop from the KDE 3.5 svn branch.

To reproduce the problem it is necessary to remove all old pcs-files from 
$KDEHOME/share/apps/kdevcppsupport and to start kdevelop with a C++ project. 
From the project options create a new pcs file for Qt. 
Set the cursor at the end of a line in the editor press enter and then any 
letter. kdevelop goes into an infinite recursive loop starting in 
languages/cpp/cppcodecompletion.cpp, line 1990.

I have modified lib/catalog/catalog.cpp with the attached patch and this 
solves the infinite recursion of kdevelop. Code completion seems to work 
however I do not have enough time right now to test the patch thoroughly and 
I have to admit that I don't know what I am doing. ;-) (Anyone daring to test 
the patch?)

As a side note: during the error hunting I had the impression that the 
pcs-files are corrupted and unusable if kdevelop is started from a console 
and stopped via Ctrl-C.

Have fun,
Jens

PS: BDB seems to be version 4.2.52

["catalog.patch" (text/x-diff)]

Index: catalog.cpp
===================================================================
--- catalog.cpp	(Revision 462077)
+++ catalog.cpp	(Arbeitskopie)
@@ -414,7 +414,7 @@
 
     cursors[ current ] = 0;
 
-    if( current > 0) {
+    if( current == args.size() ) {
 
         DBC* join_curs = 0;
         int rtn = d->dbp->join( d->dbp, cursors, &join_curs, 0 );

_______________________________________________
KDevelop-devel mailing list
KDevelop-devel@barney.cs.uni-potsdam.de
http://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic