possible bugfix in KHTMLPart::findTextNext ?

Eva Brucherseifer eva.brucherseifer at basyskom.de
Thu Feb 23 17:03:28 GMT 2006


Hi,

the method KHTMLPart::findTextNext(..) didn't work for us in our Qt2 port of 
Konq/E. With the attached patch the method works for us. 

@@ -2824,7 +2834,7 @@ bool KHTMLPart::initFindNode( bool selec
 // Old method (its API limits the available features - remove in KDE-4)
 bool KHTMLPart::findTextNext( const QString &str, bool forward, bool 
caseSensitive, bool isRegExp )
 {
-    if ( !initFindNode( false, !forward, false ) )
+    if ( !initFindNode( false, !forward, d->m_findNode ) )
       return false;
     while(1)
     {

We think, that this bugfix might also be applicable for the Qt3/Embedded and 
the desktop version of KDE 3.5. So someone here might want to check this and 
apply the patch in kdelibs.

In case you are attempted to say, that we should use the other find function:
It is more heavy-weight and therefore commented out for Embedded...

Cheers,
eva

-- 
Eva Brucherseifer
General Manager

basysKom GmbH
Robert-Bosch-Str. 7 | 64293 Darmstadt | Germany
Tel: +49 6151 3969-961 | Fax: -736 | Mobile: +49 170 5533642
eva.brucherseifer at basyskom.de | www.basyskom.de




More information about the kfm-devel mailing list