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

Michael Niedermayer michaelni at gmx.at
Thu Sep 1 03:12:49 CEST 2011


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"

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- 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/20110901/6495ba36/attachment.asc>


More information about the ffmpeg-devel mailing list