On 27/07/13 13:13, Diego Biurrun wrote: > 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. >
As you prefer. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
