[FFmpeg-cvslog] huffman: use a named identifer for the bits constant

Michael Niedermayer git at videolan.org
Tue Sep 8 14:16:57 CEST 2015


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Wed Sep  2 14:22:15 2015 +0200| [5a19bce2ff2b61602889392bec747ce81d1e9a1b] | committer: Vittorio Giovara

huffman: use a named identifer for the bits constant

Signed-off-by: Anton Khirnov <anton at khirnov.net>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=5a19bce2ff2b61602889392bec747ce81d1e9a1b
---

 libavcodec/fraps.c   |    2 +-
 libavcodec/huffman.c |    2 +-
 libavcodec/huffman.h |    1 +
 libavcodec/vp6.c     |    4 ++--
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/libavcodec/fraps.c b/libavcodec/fraps.c
index 7b19825..d502d3d 100644
--- a/libavcodec/fraps.c
+++ b/libavcodec/fraps.c
@@ -112,7 +112,7 @@ static int fraps2_decode_plane(FrapsContext *s, uint8_t *dst, int stride, int w,
     init_get_bits(&gb, s->tmpbuf, size * 8);
     for (j = 0; j < h; j++) {
         for (i = 0; i < w*step; i += step) {
-            dst[i] = get_vlc2(&gb, vlc.table, 9, 3);
+            dst[i] = get_vlc2(&gb, vlc.table, FF_HUFFMAN_BITS, 3);
             /* lines are stored as deltas between previous lines
              * and we need to add 0x80 to the first lines of chroma planes
              */
diff --git a/libavcodec/huffman.c b/libavcodec/huffman.c
index eb5f01f..a55e536 100644
--- a/libavcodec/huffman.c
+++ b/libavcodec/huffman.c
@@ -124,7 +124,7 @@ static int build_huff_tree(VLC *vlc, Node *nodes, int head, int flags)
 
     get_tree_codes(bits, lens, xlat, nodes, head, 0, 0,
                    &pos, no_zero_count);
-    return ff_init_vlc_sparse(vlc, 9, pos, lens, 2, 2, bits, 4, 4, xlat, 1, 1, 0);
+    return ff_init_vlc_sparse(vlc, FF_HUFFMAN_BITS, pos, lens, 2, 2, bits, 4, 4, xlat, 1, 1, 0);
 }
 
 
diff --git a/libavcodec/huffman.h b/libavcodec/huffman.h
index 043e6e3..3e935c0 100644
--- a/libavcodec/huffman.h
+++ b/libavcodec/huffman.h
@@ -37,6 +37,7 @@ typedef struct Node {
 
 #define FF_HUFFMAN_FLAG_HNODE_FIRST 0x01
 #define FF_HUFFMAN_FLAG_ZERO_COUNT  0x02
+#define FF_HUFFMAN_BITS 9
 
 typedef int (*HuffCmp)(const void *va, const void *vb);
 int ff_huff_build_tree(AVCodecContext *avctx, VLC *vlc, int nb_codes,
diff --git a/libavcodec/vp6.c b/libavcodec/vp6.c
index 4435444..5026c2a 100644
--- a/libavcodec/vp6.c
+++ b/libavcodec/vp6.c
@@ -402,11 +402,11 @@ static void vp6_parse_coeff_huffman(VP56Context *s)
             } else {
                 if (get_bits_left(&s->gb) <= 0)
                     return;
-                coeff = get_vlc2(&s->gb, vlc_coeff->table, 9, 3);
+                coeff = get_vlc2(&s->gb, vlc_coeff->table, FF_HUFFMAN_BITS, 3);
                 if (coeff == 0) {
                     if (coeff_idx) {
                         int pt = (coeff_idx >= 6);
-                        run += get_vlc2(&s->gb, s->runv_vlc[pt].table, 9, 3);
+                        run += get_vlc2(&s->gb, s->runv_vlc[pt].table, FF_HUFFMAN_BITS, 3);
                         if (run >= 9)
                             run += get_bits(&s->gb, 6);
                     } else



More information about the ffmpeg-cvslog mailing list