configmgr/source/valueparser.cxx |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 467eeda66ee444c846fcd89da1fe064dd06daa9d
Author:     Stephan Bergmann <stephan.bergm...@allotropia.de>
AuthorDate: Tue Feb 6 21:10:17 2024 +0100
Commit:     Stephan Bergmann <stephan.bergm...@allotropia.de>
CommitDate: Tue Feb 6 22:43:18 2024 +0100

    Fix placement of comments
    
    ...which each pertain to all the preceding code that extracts numbers from
    strings, and where 0a5d4dc25c5521de221f63dbc47c6ba79a51f8fb "elide some 
OString
    temporaries" had changed that preceding code from stretching over a single 
to
    stretching over multiple statements each
    
    Change-Id: I315187465d64f620b1ddea8a0cc74ed5f8dc113b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/163063
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de>

diff --git a/configmgr/source/valueparser.cxx b/configmgr/source/valueparser.cxx
index c81d975e474c..83ddd245dc06 100644
--- a/configmgr/source/valueparser.cxx
+++ b/configmgr/source/valueparser.cxx
@@ -94,8 +94,7 @@ bool parseValue(xmlreader::Span const & text, sal_Int16 * 
value) {
     }
     else
         n = o3tl::toInt32(std::string_view(text.begin, text.length));
-
-        //TODO: check valid lexical representation
+    //TODO: check valid lexical representation
     if (n >= SAL_MIN_INT16 && n <= SAL_MAX_INT16) {
         *value = static_cast< sal_Int16 >(n);
         return true;
@@ -140,7 +139,7 @@ bool parseValue(xmlreader::Span const & text, sal_Int64 * 
value) {
         *value = static_cast< sal_Int64 >(sSuffix.toUInt64(16));
     }
     else *value = o3tl::toInt64(std::string_view(text.begin, text.length));
-        //TODO: check valid lexical representation
+    //TODO: check valid lexical representation
     return true;
 }
 

Reply via email to