[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

Nicolas George george at nsup.org
Sun Oct 25 12:39:08 CET 2015


Le quartidi 4 brumaire, an CCXXIV, Derek Buitenhuis a écrit :
> Perhaps wm4 or Michael or someone can chime in... I don't know of a single good
> way to handle this. Just failing if the locale has a radix point as a comma instead
> of a period seems less bad than mucking with the thread/process locale inside a
> library.

Personally, I consider that users setting locale variables other than
LC_CTYPE and LC_MESSAGES mean literally "please break random programs on my
system"; I am convinced of this since more than fifteen years when initing
Gtk in the OCaml toplevel would cause "let pi = 3.14" to set pi to 3.0.

Therefore, my opinion on this is: let us document that for all FFmpeg
libraries, setting any locale except these two to anything other than
C/POSIX yields undefined behaviours.

If people want to write applications using various locales, they should use
the version with explicit locale argument introduced in the latest version
of the standard.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151025/87ac52eb/attachment.sig>


More information about the ffmpeg-devel mailing list