diff mbox series

[FFmpeg-devel,18/35] avcodec/proresenc_anatoliy: directly work with blocks in encode_dcs()

Message ID 20231211014429.1841681-19-u@pkh.me
State Accepted
Commit 273f591a3d1cc19f7d6621c9a214c69475f06b04
Headers show
Series [FFmpeg-devel,01/35] avcodec/proresenc_kostya: remove an unnecessary parenthesis level in MAKE_CODE() macro | expand

Commit Message

Clément Bœsch Dec. 11, 2023, 1:35 a.m. UTC 
This makes the function closer to encode_dcs() in proresenc_kostya.
---
 libavcodec/proresenc_anatoliy.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

Comments

Stefano Sabatini Dec. 18, 2023, 12:30 a.m. UTC | #1 
On date Monday 2023-12-11 02:35:19 +0100, Clément Bœsch wrote:
> This makes the function closer to encode_dcs() in proresenc_kostya.
> ---
>  libavcodec/proresenc_anatoliy.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

should be good
diff mbox series

Patch

diff --git a/libavcodec/proresenc_anatoliy.c b/libavcodec/proresenc_anatoliy.c
index 97d681467d..a418a12705 100644
--- a/libavcodec/proresenc_anatoliy.c
+++ b/libavcodec/proresenc_anatoliy.c
@@ -271,16 +271,17 @@  static void encode_dcs(PutBitContext *pb, int16_t *blocks,
                        int blocks_per_slice, int *qmat)
 {
     int prev_dc, codebook;
-    int i, sign, idx;
+    int i, sign;
     int new_dc, delta, diff_sign, code;
 
     prev_dc = (blocks[0] - 0x4000) / qmat[0];
     codebook = TO_GOLOMB(prev_dc);
     encode_vlc_codeword(pb, FIRST_DC_CB, codebook);
+    blocks  += 64;
 
-    codebook = 5; sign = 0; idx = 64;
-    for (i = 1; i < blocks_per_slice; i++, idx += 64) {
-        new_dc    = (blocks[idx] - 0x4000) / qmat[0];
+    codebook = 5; sign = 0;
+    for (i = 1; i < blocks_per_slice; i++, blocks += 64) {
+        new_dc    = (blocks[0] - 0x4000) / qmat[0];
         delta     = new_dc - prev_dc;
         diff_sign = DIFF_SIGN(delta, sign);
         code      = TO_GOLOMB2(get_level(delta), diff_sign);