[Libreoffice] [PATCH] new BUG 36594

Eike Rathke ooo at erack.de
Thu Sep 1 12:59:57 PDT 2011


Hi Jenei,

On Thursday, 2011-09-01 20:31:11 +0200, Jenei Gábor wrote:

> Well, as I know for HSQL -- and // are equal, so // is like one line
> comment in C++.

Hmm.. ok.

Btw, this

+        if(sCopy[i]=='\n' || i==nQueryLen-1){
+            nCommentLen++;
+            if(i==nQueryLen-1)sTemp.append(&sCopy[i],1);
+            if(bComment) bComment=false;
+            sTemp.append((sal_Unicode)'\n');

appends yet another \n if there was already one at the very end of the
query string, is that intentional?

> Well,thanks for your work on the other hand :)

I'm rearranging some logic to fold various if's away that are
unnecessarily executed in the loops. I'll let you know details later.

How can I quickly setup a database and query to test this thing?

  Eike

-- 
 PGP/OpenPGP/GnuPG encrypted mail preferred in all private communication.
 Key ID: 0x293C05FD - 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110901/061681ba/attachment.pgp>


More information about the LibreOffice mailing list