D16775: Fix using comma decimal separator in multiple argument functions

Christoph Feck noreply at phabricator.kde.org
Fri Nov 9 11:54:09 GMT 2018


cfeck requested changes to this revision.
cfeck added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> parser.cpp:1585
> +	if (m_decimalSymbol == ",")
> +		str->replace(QRegularExpression("(\\d),(\\d)"), "\\1.\\2"); //if we use comma as a decimal separator it is reasonable to separate numbers in function arguments by comma with space
> +	else

`\d` captures just a single digit.

REPOSITORY
  R334 KmPlot

REVISION DETAIL
  https://phabricator.kde.org/D16775

To: yurchor, #kde_edu, #localization, cfeck
Cc: cfeck, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181109/4187c0fa/attachment-0001.html>


More information about the kde-edu mailing list