On Tue, Apr 07, 2015 at 03:31:25AM +0530, Himangi Saraogi wrote: > --- > libavformat/format.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavformat/format.c b/libavformat/format.c > index fa94b7d..0713849 100644 > --- a/libavformat/format.c > +++ b/libavformat/format.c > @@ -172,7 +172,8 @@ AVInputFormat *av_probe_input_format3(AVProbeData *pd, > int is_opened, > AVProbeData lpd = *pd; > AVInputFormat *fmt1 = NULL, *fmt; > int score, nodat = 0, score_max = 0; > - const static uint8_t zerobuffer[AVPROBE_PADDING_SIZE]; > + static uint8_t zerobuffer[AVPROBE_PADDING_SIZE];
the buffer should never be changed, so const seems correct if its not const, a human reader might think "can this change? its not const" a cast where the warning ocurs or replacing the const by a comment seems better [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is what and why we do it that matters, not just one of them.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel