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).
> + // Skip bits: resv(1)
> + // bitrate_low(18)
> + // resv(1)
> + // bitrate_high(12)
> + // low_delay
> + skip_bits(&gb, 32);
Is "low_delay" part of the skipped bits? The rest already adds up to
32.
> + av_log(avctx, AV_LOG_DEBUG,
> + "avs3 parse seq hdr: profile %d; coded wxh: %dx%d; "
> + "frame_rate_code %d\n", profile, avctx->width,
> avctx->height, ratecode);
Incorrect indentation.
Cheers,
Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
