[Libreoffice-commits] core.git: sal/rtl

Eike Rathke erack at redhat.com
Fri Oct 20 14:42:39 UTC 2017


 sal/rtl/math.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 82a869887b46e9fc87928cf462e2a1d020d6e9c3
Author: Eike Rathke <erack at redhat.com>
Date:   Fri Oct 20 16:41:24 2017 +0200

    Comment some values that 16 doesn't work
    
    One of them was removed with
    e6611cc2ef5960e9f32c56da44fafd02446f53e6, reintroduce to prevent
    running into a dead end again.
    
    16 or 17 digits will need some different approach.
    
    Change-Id: Iec213ed857121e323e13ee83763c51fa563c794f

diff --git a/sal/rtl/math.cxx b/sal/rtl/math.cxx
index 32121b34b2f1..64b5be8bcc78 100644
--- a/sal/rtl/math.cxx
+++ b/sal/rtl/math.cxx
@@ -373,7 +373,7 @@ inline void doubleToString(typename T::String ** pResult,
         case rtl_math_StringFormat_Automatic:
         {   // E or F depending on exponent magnitude
             int nPrec;
-            if (nExp <= -15 || nExp >= 15) // #58531# was <-16, >16
+            if (nExp <= -15 || nExp >= 15)  // was <-16, >16 in ancient versions, which leads to inaccuracies
             {
                 nPrec = 14;
                 eFormat = rtl_math_StringFormat_E;
@@ -525,7 +525,7 @@ inline void doubleToString(typename T::String ** pResult,
     {
         for (int i = 0; ; i++)
         {
-            if (i < 15)
+            if (i < 15)     // was 16 in ancient versions, which leads to inaccuracies
             {
                 int nDigit;
                 if (nDigits-1 == 0 && i > 0 && i < 14)


More information about the Libreoffice-commits mailing list