Use existing rtl_math_stringToDouble

Desktop / LibreOffice - Stephan Bergmann [redhat.com] - 21 July 2021 13:38 UTC

...like it is also already done in LineParser::readDouble in sdext/source/pdfimport/wrapper/wrapper.cxx (esp. since the code should be changed to use C++17 std::from_chars once that is available in all our baselines), reverting again the introduction of rtl_str_toDouble_WithLength in b1df9c67349cf4cc5be4128d797aefb87f50e38f "[API CHANGE] reduce cost of numeric conversion"

Change-Id: If7e3a15649f80093d3407157412fd3deb3a38b12 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119318

99a1290b3f2c Use existing rtl_math_stringToDouble
include/rtl/string.h | 20 --------------------
sal/rtl/string.cxx | 8 --------
sal/util/sal.map | 5 -----
sax/source/tools/fastattribs.cxx | 4 +++-
4 files changed, 3 insertions(+), 34 deletions(-)

Upstream: cgit.freedesktop.org


  • Share