[vlc-devel] vlc: svn commit r14691 (courmisch)

Subversion daemon svn at videolan.org
Thu Mar 9 14:08:48 CET 2006

r14691 | courmisch | 2006-03-09 14:08:46 +0100 (Thu, 09 Mar 2006) | 15 lines
Changed paths:
   M /trunk/src/misc/configuration.c

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

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).

This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html

More information about the vlc-devel mailing list