On Tue, Feb 25, 2014 at 02:36:55PM +0200, Martin Storsjö wrote:
> On Tue, 25 Feb 2014, Diego Biurrun wrote:
> >--- a/libavcodec/parser.c
> >+++ b/libavcodec/parser.c
> >@@ -137,7 +137,7 @@ int av_parser_parse2(AVCodecParserContext *s,
> >AVCodecContext *avctx,
> > buf = dummy_buf;
> > } else if (s->cur_offset + buf_size !=
> > s->cur_frame_end[s->cur_frame_start_index]) { /* skip remainder packets */
> > /* add a new packet descriptor */
> >- i = (s->cur_frame_start_index + 1) & (AV_PARSER_PTS_NB - 1);
> >+ i = s->cur_frame_start_index + 1 & AV_PARSER_PTS_NB - 1;
>
> I very much prefer to keep parentheses in this case. Yes, they're
> not strictly required, but I still like to have them for clarity.
Yeah, those are kind of borderline. I'll push with this hunk dropped.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
