On date Monday 2023-12-11 02:35:05 +0100, Clément Bœsch wrote: > This is going to be shared with proresenc_kostya in the upcoming commit. > --- > libavcodec/proresdata.c | 6 ++++++ > libavcodec/proresdata.h | 2 ++ > libavcodec/proresenc_anatoliy.c | 9 ++------- > 3 files changed, 10 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/proresdata.c b/libavcodec/proresdata.c > index 4787907c71..12ee43bddd 100644 > --- a/libavcodec/proresdata.c > +++ b/libavcodec/proresdata.c > @@ -43,3 +43,9 @@ const uint8_t ff_prores_interlaced_scan[64] = { > 30, 23, 31, 38, 45, 52, 60, 53, > 46, 39, 47, 54, 61, 62, 55, 63 > }; > + > +const uint8_t ff_prores_run_to_cb[16] = { 0x06, 0x06, 0x05, 0x05, 0x04, 0x29, > + 0x29, 0x29, 0x29, 0x28, 0x28, 0x28, 0x28, 0x28, 0x28, 0x4C }; > +
> +const uint8_t ff_prores_lev_to_cb[10] = { 0x04, 0x0A, 0x05, 0x06, 0x04, 0x28, > + 0x28, 0x28, 0x28, 0x4C }; while at it this could employ "lev" => "level" to clarify code and maybe enable some pattern matching trick > diff --git a/libavcodec/proresdata.h b/libavcodec/proresdata.h > index d8c8786689..4d16992892 100644 > --- a/libavcodec/proresdata.h > +++ b/libavcodec/proresdata.h > @@ -29,6 +29,8 @@ > > extern const uint8_t ff_prores_progressive_scan[64]; > extern const uint8_t ff_prores_interlaced_scan[64]; > +extern const uint8_t ff_prores_run_to_cb[16]; > +extern const uint8_t ff_prores_lev_to_cb[10]; > > #define FIRST_DC_CB 0xB8 // rice_order = 5, exp_golomb_order = 6, > switch_bits = 0 > > diff --git a/libavcodec/proresenc_anatoliy.c b/libavcodec/proresenc_anatoliy.c > index e7114f1646..6b2e513bed 100644 > --- a/libavcodec/proresenc_anatoliy.c > +++ b/libavcodec/proresenc_anatoliy.c > @@ -296,11 +296,6 @@ static void encode_dc_coeffs(PutBitContext *pb, int16_t > *in, > } > } > > -static const uint8_t run_to_cb[16] = { 0x06, 0x06, 0x05, 0x05, 0x04, 0x29, > - 0x29, 0x29, 0x29, 0x28, 0x28, 0x28, 0x28, 0x28, 0x28, 0x4C }; > -static const uint8_t lev_to_cb[10] = { 0x04, 0x0A, 0x05, 0x06, 0x04, 0x28, > - 0x28, 0x28, 0x28, 0x4C }; > - > static void encode_ac_coeffs(PutBitContext *pb, > int16_t *in, int blocks_per_slice, int *qmat, const uint8_t > ff_prores_scan[64]) > { > @@ -313,14 +308,14 @@ static void encode_ac_coeffs(PutBitContext *pb, > for (j = 0; j < blocks_per_slice; j++) { > int val = QSCALE(qmat, indp, in[(j << 6) + indp]); > if (val) { > - encode_codeword(pb, run, run_to_cb[FFMIN(prev_run, 15)]); > + encode_codeword(pb, run, ff_prores_run_to_cb[FFMIN(prev_run, > 15)]); > > prev_run = run; > run = 0; > level = get_level(val); > code = level - 1; > > - encode_codeword(pb, code, lev_to_cb[FFMIN(prev_level, 9)]); > + encode_codeword(pb, code, > ff_prores_lev_to_cb[FFMIN(prev_level, 9)]); > > prev_level = level; > > -- > 2.43.0 > > _______________________________________________ > 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". _______________________________________________ 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".