[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