[Paraview] [Paraview-developers] Locale settings in PV

[Cory Quammen]

Hi Gerald,

Thanks for your report and fix.

It looks like pqDoubleEdit is used only in the PointSprite plugin. Are you
seeing this problem only in the PointSprite plugin, or do you see problems
more widely in ParaView?

Thanks,
Cory

On Thu, Oct 15, 2015 at 5:46 AM, Lodron, Gerald <Gerald.Lodron at joanneum.at>
wrote:

> Hello
>
> I noticed a bug in paraview on parsing double values in QLineEdits, e.g.
> in pqDoubleEdit:
>
> double pqDoubleEdit::value()
> {
>   QString currentText = this->text();
>   int currentPos = this->cursorPosition();
>   QDoubleValidator dvalidator(NULL);
>   QValidator::State state = dvalidator.validate(currentText, currentPos);
>   if (state == QValidator::Acceptable || state == QValidator::Intermediate)
>     {
>     return currentText.toDouble();
>     }
>   return 0.0;
> }
>
> The Problem is that QDoubleValidator uses the current locale. If the
> current locale is e.g. German that e.g. “0.1” is not allowed, only “0,1”.
> On the other side the QString function toDouble cannot parse it correctly,
> see documentation *http://doc.qt.io/qt-5/qstring.html#toDouble*
> <http://doc.qt.io/qt-5/qstring.html#toDouble> , here the
> *http://doc.qt.io/qt-5/qlocale.html#toDouble*
> <http://doc.qt.io/qt-5/qlocale.html#toDouble> is recommended…. So all
> QLineEdit functions with toDouble must be replaced by
> QLocale oL;
> Double dValue = oL.toDouble( lineEdit->text())
>
> Nasty….Other solution would be to set the locale to English for whole
> paraview regardless to the system locale, but I don’t know if that is
> possible (also not very beautiful)…
>
> Best regards
>
>
>
> *------------------------------------------------------------------------------------*
> *Gerald Lodron*
>
> Researcher of Machine Vision Applications Group
> DIGITAL - Institute for Information and Communication Technologies
>
> JOANNEUM RESEARCH Forschungsgesellschaft mbH
> Steyrergasse 17, 8010 Graz, AUSTRIA
>
> phone:   +43-316-876-1751
> general fax: +43-316-876-1751
> web: *http://www.joanneum.at/digital* <http://www.joanneum.at/digital>
> e-mail: *gerald.lodron at joanneum.at* <gerald.lodron at joanneum.at>
>
>
>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Search the list archives at:
> http://markmail.org/search/?q=Paraview-developers
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/paraview-developers
>
>


-- 
Cory Quammen
R&D Engineer
Kitware, Inc.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview/attachments/20151016/ca87c0dd/attachment.html>