[Libreoffice-commits] core.git: ucb/source

Chr. Rossmanith ChrRossmanith at gmx.de
Sat Apr 6 07:51:25 PDT 2013


 ucb/source/regexp/regexp.cxx |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit a07505865b624b9c9fc5a33e7604ba982302c794
Author: Chr. Rossmanith <ChrRossmanith at gmx.de>
Date:   Sat Apr 6 16:50:34 2013 +0200

    Add comment the RTL_CONSTASCII_STRINGPARAM should be kept in regexp.cxx
    
    Change-Id: I3cd2129d98c3d936d8ec31f89554b37f12427c1c

diff --git a/ucb/source/regexp/regexp.cxx b/ucb/source/regexp/regexp.cxx
index 58ba065..ffa318c 100644
--- a/ucb/source/regexp/regexp.cxx
+++ b/ucb/source/regexp/regexp.cxx
@@ -374,6 +374,9 @@ Regexp Regexp::parse(rtl::OUString const & rRegexp)
     if (p == pEnd)
         throw lang::IllegalArgumentException();
 
+    // This and the matchString() calls below are some of the few places where
+    // RTL_CONSTASCII_STRINGPARAM() should NOT be removed.
+    // (c.f. https://gerrit.libreoffice.org/3117)
     if (matchString(&p, pEnd, RTL_CONSTASCII_STRINGPARAM(".*")))
     {
         if (p != pEnd)


More information about the Libreoffice-commits mailing list