[MPlayer-dev-eng] [PATCH] implement the setlocale feature
Ivan Kalvachev
ikalvachev at gmail.com
Sun Jun 18 23:02:24 CEST 2006
2006/6/18, Fengguang Wu <fengguang.wu at gmail.com>:
> Greetings,
>
> The --enable-iconv and --enable-langinfo features take no effect now,
> since the setlocale stuffs have been missing.
>
> The attached patch completes the feature set:
> - implement the missing --disable-setlocale handler
> - call setlocale(LC_ALL, "") if defined USE_SETLOCALE
configure changes looks ok.
About asf_mmst_streaming.c :
1. The #undef ICONV really should not be there.
2. What kind of bug is the setlocale() change fixing? Doesn't it beat
the purpose of using nl_langinfo()??
3. Is the iconv_open() change only adding one space? Such kind of
change is considered cosmetics and is not allowed.
More information about the MPlayer-dev-eng
mailing list