Bug 261884

Summary:	When importing a file .pov with ", " as decimal separator all the digits are truncated at the integer part.
Product:	[Unmaintained] kpovmodeler	Reporter:	Marco Bobba <bobba.marco>
Component:	general	Assignee:	Andreas Zehender <zehender>
Status:	RESOLVED FIXED
Severity:	normal	CC:	cfeck
Priority:	NOR
Version:	CVS
Target Milestone:	---
Platform:	Debian unstable
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Set LC_NUMERIC to POSIX during the conversion

Description Marco Bobba 2011-01-02 18:21:01 UTC

Version:           CVS (using KDE 4.4.5) 
OS:                Linux

atof convert ascii to float using the locale setting but povray need "." as decimal separator. In system where the decimal separator is "," the value are truncated at the integer part.
This patch seem work ...

Reproducible: Always

Steps to Reproduce:
Set a locale with "," as decimal separator and  import a file .pov with float values

Actual Results:  
Integer truncated

Expected Results:  
float

Comment 1 Marco Bobba 2011-01-02 18:22:18 UTC

Created attachment 55476 [details]
Set LC_NUMERIC to POSIX during the conversion

Comment 2 Christoph Feck 2011-01-03 02:07:34 UTC

Right, and this was the issue you fixed in your original patch by replacing atof with a Qt function. I think replacing it is a better fix. Will recheck your old patch in the coming days.

Comment 3 Christoph Feck 2011-01-08 19:05:59 UTC

SVN commit 1212937 by cfeck:

Parse floats in C locale

Use QByteArray::toDouble() instead of atof(). The Qt function
is documented to always use the C locale (which has "." as the
decimal separator), instead of the current locale.

BUG: 261884


 M  +2 -2      pmscanner.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1212937