Configuration uses %f

%f printf() specifier gives inconsistent result depending on the user's locale. We cannot assume it will be the american format either (as per #581 (closed)). You cannot use strtof() either.

Don't use it when read from/writing to a file or a network stream.

In any case, whether we fix or don't fix this bug, it will cause problems, so we should fix it ASAP.

added Component::Core Severity::normal Type::bug Version::master git + 1 deleted label

changed milestone to %0.8.5-test3

changed the description

assigned to @Courmisch and unassigned @MigrationBot

(In [14691]) As a work-around, accept both decimal separators when... deserializing floats from configuration (fixes #589 (closed))

I know this is NOT the ?\226?\128?\156right?\226?\128?\157 way to fix this. The ?\226?\128?\156right?\226?\128?\157 way would be to remove floats from the configuration... but there are approximately one float parameter in the core and 46 ones in our current plugins set, none of which I use myself. So I am defintely not going to rewrite all^H^H^Hany of them.

Of course, it would be cleaner to be ?\226?\128?\156strict?\226?\128?\157 in what we write to the configuration file, than ?\226?\128?\156liberal?\226?\128?\157 in what we accept... but IMHO, it is way easier and more robust to be liberal in what we accept in this particular case (ie. accepting both decimal separators is trivial, while serializing float numbers manually is complex).

added Status::fixed label

closed