On Tue, 25 Feb 2014, Diego Biurrun wrote:
---Missed two lines before .. libavcodec/parser.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/parser.c b/libavcodec/parser.c index c6ceeb6..d44ecb0 100644 --- 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.
// Martin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
