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

List:       koffice-devel
Subject:    koffice/kspread/dialogs
From:       Adriaan de Groot <groot () kde ! org>
Date:       2005-06-30 14:02:03
Message-ID: 1120140123.927577.31248.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 430209 by adridg:

If you want \s in a RE to represent a whitespace character, 
be careful about C's character escaping as well. Need
\\s in the source, which gives \s in the string. Caught by
gcc 2.95 but not by later gcc's.

CCMAIL: koffice-devel@kde.org


 M  +2 -2      kspread_dlg_database.cc  


--- trunk/koffice/kspread/dialogs/kspread_dlg_database.cc #430208:430209
@@ -600,8 +600,8 @@
   // An update command must also be followed by a space, or it would be parsed
   // as an identifier.
   // For sanity, also check that there is a SELECT 
-  QRegExp couldModifyDB( "(^|[( \s])(UPDATE|DELETE|INSERT|CREATE) ", false /* cs */ );
-  QRegExp couldQueryDB( "(^|[( \s])(SELECT) ", false /* cs */ );
+  QRegExp couldModifyDB( "(^|[( \\s])(UPDATE|DELETE|INSERT|CREATE) ", false /* cs */ );
+  QRegExp couldQueryDB( "(^|[( \\s])(SELECT) ", false /* cs */ );
 
   if (couldModifyDB.search( queryStr ) != -1 || couldQueryDB.search ( queryStr ) == -1 )
   {
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]