[SCM] libav/experimental: golomb: Fix the implementation of get_se_golomb_long

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Aug 10 16:02:29 UTC 2014


The following commit has been merged in the experimental branch:
commit 508a84e6726ab94a740c160b30fd8162265d1fef
Author: Martin Storsjö <martin at martin.st>
Date:   Sat Mar 29 12:35:11 2014 +0200

    golomb: Fix the implementation of get_se_golomb_long
    
    This was only used in hevc muxing code so far.
    
    This makes the return values match what get_se_golomb returns for
    the same bitstream reader instances.
    
    The logic for producing a signed golomb code out of an unsigned one
    was based on the corresponding code in get_se_golomb, which operated
    directly on the bitstream reader buffer - not on the equivalent
    return value from get_ue_golomb.
    
    CC: libav-stable at libav.org
    Signed-off-by: Martin Storsjö <martin at martin.st>

diff --git a/libavcodec/golomb.h b/libavcodec/golomb.h
index ce3500f..1754706 100644
--- a/libavcodec/golomb.h
+++ b/libavcodec/golomb.h
@@ -211,9 +211,9 @@ static inline int get_se_golomb_long(GetBitContext *gb)
     unsigned int buf = get_ue_golomb_long(gb);
 
     if (buf & 1)
-        buf = -(buf >> 1);
+        buf = (buf + 1) >> 1;
     else
-        buf = (buf >> 1);
+        buf = -(buf >> 1);
 
     return buf;
 }

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list