starmath/source/parse.cxx |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 2b2d95de7d753b2f673d6fed1e5dc47fc36c8a3b
Author:     Eike Rathke <er...@redhat.com>
AuthorDate: Wed Oct 2 12:41:07 2019 +0200
Commit:     Xisco FaulĂ­ <xiscofa...@libreoffice.org>
CommitDate: Thu Oct 3 11:02:30 2019 +0200

    Resolves: tdf#127873 accept ',' comma group separator in number entities 
again
    
    Existing documents unwittingly may have used that as decimal
    separator in such locales (though it never was as this is always
    the en-US locale).
    
    "Regression" from
    
    commit 9336286a7ea5385541344f444e6f8702c85bdacb
    CommitDate: Fri Nov 30 22:15:22 2018 +0100
    
        [API CHANGE] Resolves: tdf#42518 new 
KParseTokens::GROUP_SEPARATOR_IN_NUMBER
    
    Change-Id: I0ffc1b8ec7c1820fccd8277036c64093dddf82fe
    Reviewed-on: https://gerrit.libreoffice.org/80023
    Reviewed-by: Eike Rathke <er...@redhat.com>
    Tested-by: Jenkins
    (cherry picked from commit 2caa9d7ddcff3f6e380c306b737e9b5e9cdaf4c2)
    Reviewed-on: https://gerrit.libreoffice.org/80045
    Reviewed-by: Xisco FaulĂ­ <xiscofa...@libreoffice.org>

diff --git a/starmath/source/parse.cxx b/starmath/source/parse.cxx
index 3cd64af729db..90e356505cbf 100644
--- a/starmath/source/parse.cxx
+++ b/starmath/source/parse.cxx
@@ -376,8 +376,13 @@ void SmParser::NextToken()
         KParseTokens::IGNORE_LEADING_WS;
 
     // Continuing characters for numbers, may be any numeric or dot.
+    // tdf#127873: additionally accept ',' comma group separator as too many
+    // existing documents unwittingly may have used that as decimal separator
+    // in such locales (though it never was as this is always the en-US locale
+    // and the group separator is only parsed away).
     static const sal_Int32 coNumContFlags =
-        coNumStartFlags & ~KParseTokens::IGNORE_LEADING_WS;
+        (coNumStartFlags & ~KParseTokens::IGNORE_LEADING_WS) |
+        KParseTokens::GROUP_SEPARATOR_IN_NUMBER;
 
     sal_Int32   nBufLen = m_aBufferString.getLength();
     ParseResult aRes;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to