[FFmpeg-devel] [PATCH] Replace all strcasecmp/strncasecmp usages.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Thu Nov 3 19:27:46 CET 2011


On Wed, Nov 02, 2011 at 09:04:03PM +0100, Reimar Döffinger wrote:
> On Wed, Nov 02, 2011 at 08:57:08PM +0100, Michael Niedermayer wrote:
> > 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
> 
> Since it touches a lot of code by other people and I have not closely
> looked at the individual changes I'd welcome if some more people
> could give it a bit of a look.
> Otherwise I'll probably apply tomorrow.

Pushed.


More information about the ffmpeg-devel mailing list