[FFmpeg-devel] [PATCH 2/2] cmdutils: fix uninitialized variable (type) warning.

Clément Bœsch ubitux at gmail.com
Thu Sep 1 20:34:33 CEST 2011


On Thu, Sep 01, 2011 at 03:12:49AM +0200, Michael Niedermayer wrote:
> On Thu, Sep 01, 2011 at 02:57:18AM +0200, Clément Bœsch wrote:
> > ---
> >  cmdutils.c |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> > 
> > diff --git a/cmdutils.c b/cmdutils.c
> > index d42a6e8..3359d22 100644
> > --- a/cmdutils.c
> > +++ b/cmdutils.c
> > @@ -805,6 +805,7 @@ int check_stream_specifier(AVFormatContext *s, AVStream *st, const char *spec)
> >          case 'a': type = AVMEDIA_TYPE_AUDIO;    break;
> >          case 's': type = AVMEDIA_TYPE_SUBTITLE; break;
> >          case 'd': type = AVMEDIA_TYPE_DATA;     break;
> > +        default: abort();
> >          }
> 
> i dont mind, but this is unreachable, maybe add a
> "// to prevent warning"
> 

Pushed with a comment.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110901/808bccf5/attachment.asc>


More information about the ffmpeg-devel mailing list