On date Sunday 2024-01-07 19:16:47 +0100, Clément Bœsch wrote:
> The layout for the frame flags is as follow:
>
>chroma_format u(2)
>reserved u(2)
>interlace_mode u(2)
>reserved u(2)
>
> chroma_format has 2 allowed values:
>0: reserved
>1: reserved
>2: 4:2:2
>3: 4:4:4
>
> interlace_mode has 3 allowed values:
>0: progressive
>1: tff
>2: bff
>3: reserved
>
> 0x80 is what we expect for "422 not interlaced", and the extra 0x2 from
> 0x82 is actually writting into the reserved bits.
nit: writting typo
> ---
> libavcodec/proresenc_anatoliy.c | 2 +-
> tests/ref/vsynth/vsynth1-prores | 2 +-
> tests/ref/vsynth/vsynth1-prores_444 | 2 +-
> tests/ref/vsynth/vsynth1-prores_444_int | 2 +-
> tests/ref/vsynth/vsynth1-prores_int | 2 +-
> tests/ref/vsynth/vsynth2-prores | 2 +-
> tests/ref/vsynth/vsynth2-prores_444 | 2 +-
> tests/ref/vsynth/vsynth2-prores_444_int | 2 +-
> tests/ref/vsynth/vsynth2-prores_int | 2 +-
> tests/ref/vsynth/vsynth3-prores | 2 +-
> tests/ref/vsynth/vsynth3-prores_444 | 2 +-
> tests/ref/vsynth/vsynth3-prores_444_int | 2 +-
> tests/ref/vsynth/vsynth3-prores_int | 2 +-
> tests/ref/vsynth/vsynth_lena-prores | 2 +-
> tests/ref/vsynth/vsynth_lena-prores_444 | 2 +-
> tests/ref/vsynth/vsynth_lena-prores_444_int | 2 +-
> tests/ref/vsynth/vsynth_lena-prores_int | 2 +-
> 17 files changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/libavcodec/proresenc_anatoliy.c b/libavcodec/proresenc_anatoliy.c
> index 1112cb26f1..bee4a14181 100644
> --- a/libavcodec/proresenc_anatoliy.c
> +++ b/libavcodec/proresenc_anatoliy.c
> @@ -769,7 +769,7 @@ static int prores_encode_frame(AVCodecContext *avctx,
> AVPacket *pkt,
> bytestream_put_buffer(, ctx->vendor, 4);
> bytestream_put_be16(, avctx->width);
> bytestream_put_be16(, avctx->height);
> -frame_flags = 0x82; /* 422 not interlaced */
> +frame_flags = 0x80; /* 422 not interlaced */
LGTM, thanks.
___
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".