[bt-devel] Some patches

jdc jdc.email at gmail.com
Sun Mar 19 17:11:36 MST 2006



csearchdialog.cpp

I installed an update for my SuSE 10.0.  Now the button for Yes to index returns 
259 which is the logical or of Yes and Default.

in function startSearch()
approx line 116

if ((result == (QMessageBox::Yes | QMessageBox::Default)) || (result == 
QMessageBox::Yes) || (result == QMessageBox::Default) ) {




csearchdialogpages.cpp

I found that the Strong's numbers were being matched to shorter numbers.
Like H012 would match H012X and H012XX.

in function getStrongsNumberText()
approx line 119
       // REPLACE-> if (sNumber.find(lemmaText) >= 0) {
       if (sNumber == lemmaText) {

in function highlightSearchedText()
approx line 280
          // REPLACE-> if (lemmaText.find(sNumber) >= 0) {
          if (sNumber == lemmaText) {



Next I found that words like AND and OR were being highlighted when AND and OR 
are operators to the search function.

in the function highlightSearchedText()
approx line 310
	QStringList words = QStringList::split(" ", searchedText);
	for ( int wi = 0; (unsigned int)wi < words.count(); ++wi ) {
		QString word = words[ wi ];
-> ADD		if ((word == "AND") || (word == "OR"))
-> ADD			continue;


There are a few other things broken with the highlightSearchedText.
	1) "quoted strings" are not treated as a single token.
	2) small words are highlighted within big words.

Would you like me to take a stab at fixing these?

Thanks
Jim


More information about the bt-devel mailing list