On Thu, Jul 25, 2013 at 06:08:03PM +0200, Luca Barbato wrote: > On 25/07/13 14:06, Diego Biurrun wrote: > > --- a/libavformat/flvdec.c > > +++ b/libavformat/flvdec.c > > @@ -125,8 +125,6 @@ static int flv_same_audio_codec(AVCodecContext *acodec, > > int flags) > > default: > > return acodec->codec_tag == (flv_codecid >> > > FLV_AUDIO_CODECID_OFFSET); > > } > > - > > - return 0; > > } > > Those above are safer to be left.
There's a big switch with a return in every case. If another case w/o return is added, there will be a warning. I don't see a problem. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
