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

Stephan Bergmann sbergman at redhat.com
Mon Apr 1 23:49:13 PDT 2013


 l10ntools/source/xmlparse.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit a47e50e3386a13a1601b3aa1355de70aa9e056ff
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Apr 2 08:47:27 2013 +0200

    Prevent asserts from firing
    
    ...whether or not this is already the correct fix.  The twisted
    nEndPos/nStartPos arguments in the calls to lcl_QuotRange look somewhat odd.
    
    Change-Id: If22afb058012c830b5aa6d059c7df0b4e9c53696

diff --git a/l10ntools/source/xmlparse.cxx b/l10ntools/source/xmlparse.cxx
index c76d5a6..651a27a 100644
--- a/l10ntools/source/xmlparse.cxx
+++ b/l10ntools/source/xmlparse.cxx
@@ -1180,8 +1180,8 @@ static icu::UnicodeString lcl_QuotRange(
     const sal_Int32 nEnd, bool bInsideTag = false )
 {
     icu::UnicodeString sReturn;
-    assert( nStart > 0 && nStart < rString.length() );
-    assert( nEnd > 0 && nEnd < rString.length() );
+    assert( nStart >= 0 );
+    assert( nEnd < rString.length() );
     for (sal_Int32 i = nStart; i <= nEnd; ++i)
     {
         switch (rString[i])


More information about the Libreoffice-commits mailing list