<div dir="ltr">Hi Gerald,<div><br></div><div>Thanks for your report and fix.</div><div><br></div><div>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?</div><div><br></div><div>Thanks,</div><div>Cory</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 15, 2015 at 5:46 AM, Lodron, Gerald <span dir="ltr"><<a href="mailto:Gerald.Lodron@joanneum.at" target="_blank">Gerald.Lodron@joanneum.at</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">






<div>
<font face="Calibri" size="2"><span style="font-size:11pt">
<div>Hello</div>
<div> </div>
<div>I noticed a bug in paraview on parsing double values in QLineEdits, e.g. in pqDoubleEdit:</div>
<div> </div>
<div>double pqDoubleEdit::value()</div>
<div>{</div>
<div>  QString currentText = this->text();</div>
<div>  int currentPos = this->cursorPosition();</div>
<div>  QDoubleValidator dvalidator(NULL);</div>
<div>  QValidator::State state = dvalidator.validate(currentText, currentPos);</div>
<div>  if (state == QValidator::Acceptable || state == QValidator::Intermediate)</div>
<div>    {</div>
<div>    return currentText.toDouble();</div>
<div>    }</div>
<div>  return 0.0;</div>
<div>}</div>
<div> </div>
<div>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 <a href="http://doc.qt.io/qt-5/qstring.html#toDouble" target="_blank"><font color="blue"><u>http://doc.qt.io/qt-5/qstring.html#toDouble</u></font></a>
, here the <a href="http://doc.qt.io/qt-5/qlocale.html#toDouble" target="_blank"><font color="blue"><u>http://doc.qt.io/qt-5/qlocale.html#toDouble</u></font></a> is recommended…. So all QLineEdit functions with toDouble must be replaced by </div>
<div>QLocale oL;</div>
<div>Double dValue = oL.toDouble( lineEdit->text())</div>
<div> </div>
<div>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)…</div>
<div> </div>
<div>Best regards</div>
<div> </div>
<div> </div>
<div><b>------------------------------------------------------------------------------------</b></div>
<div><b>Gerald Lodron</b></div>
<div><font size="1"><span style="font-size:7.5pt"> </span></font></div>
<div><font color="gray">Researcher of Machine Vision Applications Group</font></div>
<div><font color="gray">DIGITAL - Institute for Information and Communication Technologies</font></div>
<div> </div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">JOANNEUM RESEARCH Forschungsgesellschaft mbH</span></font></div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">Steyrergasse 17, 8010 Graz, AUSTRIA</span></font></div>
<div> </div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">phone:   <a href="tel:%2B43-316-876-1751" value="+433168761751" target="_blank">+43-316-876-1751</a>   </span></font></div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">general fax: <a href="tel:%2B43-316-876-1751" value="+433168761751" target="_blank">+43-316-876-1751</a></span></font></div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">web: <a href="http://www.joanneum.at/digital" target="_blank"><u>http://www.joanneum.at/digital</u></a></span></font></div>
<div><font size="1" color="gray"><span style="font-size:7.5pt">e-mail: <a href="mailto:gerald.lodron@joanneum.at" target="_blank"><font color="blue"><u>gerald.lodron@joanneum.at</u></font></a></span></font></div>
<div> </div>
<div> </div>
<div> </div>
</span></font>
</div>

<br>_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=Paraview-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=Paraview-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/paraview-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/paraview-developers</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cory Quammen<br>R&D Engineer<br>Kitware, Inc.</div>
</div>