Poco::NumberParser::tryParseHex raise Poco::SyntaxException exception.

[uzanka]

Describe the bug
The following functions raise Poco::SyntaxException exceptions:
Poco::NumberParser::tryParseHex, ... (all tryParse* ?)

Expected behavior
tryParse* functions does not raise any exceptions.

Logs
NumericString.h : strToInt

		if (*pStr == thSep)
		{
			if (base == 10) continue;
			throw Poco::SyntaxException("strToInt: thousand separators only allowed for base 10");
		}

Please add relevant environment information:

• OS: Windows11
• POCO Version: 1.12, or later
  (The problem does not occur in version 1.9.)

[matejk]

@uzanka , does the problem occur with the latest version also (1.14.0)?

[uzanka]

The problem also occurs in 1.14.1

Sample code and results.

#include <iostream>
#include "Poco/NumberParser.h"
#include "Poco/Exception.h"

int main() {
    try {
      unsigned int value = 0;
      bool result = Poco::NumberParser::tryParseHex("0,2", value);
      std::cout << result << std::endl;
      std::cout << value << std::endl;
      return 0;
    } catch (Poco::Exception& exc) {
      std::cout << exc.displayText() << std::endl;
      return 1;
    }
}

D:\test\ConsoleApplication2\x64\Debug>ConsoleApplication2.exe
Syntax error: strToInt: thousand separators only allowed for base 10

D:\test\ConsoleApplication2\x64\Debug>echo %ERRORLEVEL%
1

[ssooffiiaannee]

Why "Expected behavior tryParse* functions does not raise any exceptions" ? isn't this an expected behavior ? hex shall not expect commas

[uzanka]

Why do you think the string you give us is just a valid hexadecimal number?

Why do you think there are two types of parse and tryParse?

If tryParse raises an exception, then parse is enough. No need for tryParse.