Re: [FFmpeg-devel] [PATCH v5 3/5] lavc/avs3_parser: add avs3 parser

2020-08-26 Thread hwren
At 2020-08-26 19:51:20, "Moritz Barsnick" wrote: >On Thu, Aug 20, 2020 at 13:31:12 +0800, hwren wrote: >> At 2020-08-19 22:14:04, "Moritz Barsnick" wrote: >> >ffmpeg prefers the "cur++" style (twice in this block, and elsewhere). >> >> Will be corrected. Thanks. > >> >Is

Re: [FFmpeg-devel] [PATCH v5 3/5] lavc/avs3_parser: add avs3 parser

2020-08-26 Thread Moritz Barsnick
On Thu, Aug 20, 2020 at 13:31:12 +0800, hwren wrote: > At 2020-08-19 22:14:04, "Moritz Barsnick" wrote: > >ffmpeg prefers the "cur++" style (twice in this block, and elsewhere). > > Will be corrected. Thanks. > >Is "low_delay" part of the skipped bits? The rest already adds up to > >32. > > No

Re: [FFmpeg-devel] [PATCH v5 3/5] lavc/avs3_parser: add avs3 parser

2020-08-19 Thread hwren
At 2020-08-19 22:14:04, "Moritz Barsnick" wrote: >On Wed, Aug 19, 2020 at 14:25:54 +0800, hwr...@126.com wrote: >> +for (; cur < buf_size; ++cur) { >> +state = (state << 8) | buf[cur]; >> +if (ISPIC(buf[cur])){ >> +++cur; >

Re: [FFmpeg-devel] [PATCH v5 3/5] lavc/avs3_parser: add avs3 parser

2020-08-19 Thread Moritz Barsnick
On Wed, Aug 19, 2020 at 14:25:54 +0800, hwr...@126.com wrote: > +for (; cur < buf_size; ++cur) { > +state = (state << 8) | buf[cur]; > +if (ISPIC(buf[cur])){ > +++cur; ffmpeg prefers the "cur++" style (twice in this block, and elsewhere). > +

[FFmpeg-devel] [PATCH v5 3/5] lavc/avs3_parser: add avs3 parser

2020-08-19 Thread hwrenx
From: hwren Signed-off-by: hbj Signed-off-by: hwren --- libavcodec/Makefile | 1 + libavcodec/avs3_parser.c | 180 +++ libavcodec/parsers.c | 1 + 3 files changed, 182 insertions(+) create mode 100644 libavcodec/avs3_parser.c diff --git