[FFmpeg-devel] [PATCH] Replace all strcasecmp/strncasecmp usages.
Michael Niedermayer
michaelni at gmx.at
Wed Nov 2 20:57:08 CET 2011
On Wed, Nov 02, 2011 at 08:26:19PM +0100, Reimar Döffinger wrote:
> All current usages of it are incompatible with localization.
> For example strcasecmp("i", "I") != 0 is possible, but would
> break many of the places where it is used.
> Instead use our own implementations that always treat the data
> as ASCII.
>
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
> libavdevice/v4l2.c | 4 ++--
> libavfilter/libmpcodecs/vf_palette.c | 20 ++++++++++----------
> libavformat/avidec.c | 4 ++--
> libavformat/http.c | 17 ++++++++---------
> libavformat/img2.c | 7 +++----
> libavformat/matroskaenc.c | 4 ++--
> libavformat/metadata.c | 6 +++---
> libavformat/mp3enc.c | 3 +--
> libavformat/nutdec.c | 5 ++---
> libavformat/rtpdec.c | 4 ++--
> libavformat/rtpdec_mpeg4.c | 3 +--
> libavformat/rtsp.c | 11 +++++------
> libavformat/utils.c | 7 +++----
> libavutil/avstring.c | 27 +++++++++++++++++++++++++++
> libavutil/avstring.h | 12 ++++++++++++
> libavutil/avutil.h | 2 +-
> libavutil/internal.h | 4 ++++
> libavutil/parseutils.c | 7 +++----
> 18 files changed, 91 insertions(+), 56 deletions(-)
LGTM, thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111102/0aa4335d/attachment.asc>
More information about the ffmpeg-devel
mailing list