By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

[patch] kdepim 3.5.13 : optimize the "addQuotes" and "removeQuots" functions to avoid insanely long response time in large Imap folders

kdepim-3.5.13-fix_kio_imap4_addquotes.patch (text/plain), 1.47 KB, created by Francois Andriot on 2012-05-29 13:44:08 CDT

(hide)

Description: kdepim 3.5.13 : optimize the "addQuotes" and "removeQuots" functions to avoid insanely long response time in large Imap folders

Filename:

MIME Type:

Creator: Francois Andriot

Created: 2012-05-29 13:44:08 CDT

Size: 1.47 KB

patch

obsolete

>--- kdepim/libkmime/kmime_util.cpp.addquotes 2012-05-28 22:58:31.257167175 +0200 >+++ kdepim/libkmime/kmime_util.cpp 2012-05-29 20:02:19.679233274 +0200 >@@ -436,49 +436,26 @@ > > void removeQuots(TQCString &str) > { >- bool inQuote=false; >- >- for (int i=0; i < (int)str.length(); i++) { >- if (str[i] == '"') { >- str.remove(i,1); >- i--; >- inQuote = !inQuote; >- } else { >- if (inQuote && (str[i] == '\\')) >- str.remove(i,1); >- } >- } >+ str.replace(TQRegExp("\\\""), "\""); >+ str.replace(TQRegExp("\\\\"), "\\"); > } > > > void removeQuots(TQString &str) > { >- bool inQuote=false; >- >- for (int i=0; i < (int)str.length(); i++) { >- if (str[i] == '"') { >- str.remove(i,1); >- i--; >- inQuote = !inQuote; >- } else { >- if (inQuote && (str[i] == '\\')) >- str.remove(i,1); >- } >- } >+ str.replace(TQRegExp("\\\""), "\""); >+ str.replace(TQRegExp("\\\\"), "\\"); > } > > > void addQuotes(TQCString &str, bool forceQuotes) > { > bool needsQuotes=false; >- for (unsigned int i=0; i < str.length(); i++) { >- if (strchr("()<>@,.;:[]=\\\"",str[i])!=0) >- needsQuotes = true; >- if (str[i]=='\\' || str[i]=='\"') { >- str.insert(i, '\\'); >- i++; >- } >- } >+ if ( TQString( str ) .contains( TQRegExp( TQString( "\"|\\\\|=|\\]|\\[|:|;|,|\\.|,|@|<|>|\\)|\\(" ) ) ) ) >+ needsQuotes = true; >+ >+ str.replace(TQRegExp("\\"), "\\\\"); >+ str.replace(TQRegExp("\""), "\\\""); > > if (needsQuotes || forceQuotes) { > str.insert(0,'\"'); View Attachment As Diff View Attachment As Raw

• • Home
  • | New
  • | Browse
  • | Search
  • |
    [?]
  • | Reports
  • | Help
  • | New Account
  • | Log In
    [x]
  • | Forgot Password
    Login: [x]