diff options
| author | Stephan Bergmann <stephan.bergmann@allotropia.de> | 2024-02-06 21:10:17 +0100 |
|---|---|---|
| committer | Stephan Bergmann <stephan.bergmann@allotropia.de> | 2024-02-06 22:43:18 +0100 |
| commit | 467eeda66ee444c846fcd89da1fe064dd06daa9d (patch) | |
| tree | e07172b383dc7771c017074cc41003dcf849d5b6 /configmgr | |
| parent | 8f5368185661eefcd5e6e6696ac429022e0a983c (diff) | |
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.bergmann@allotropia.de>
Diffstat (limited to 'configmgr')
| -rw-r--r-- | configmgr/source/valueparser.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
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; } |