]> granicus.if.org Git - libx264/commitdiff
Fix CABAC+PCM, regression in r1592
authorFiona Glaser <fiona@x264.com>
Wed, 26 May 2010 00:49:07 +0000 (17:49 -0700)
committerFiona Glaser <fiona@x264.com>
Wed, 26 May 2010 09:06:34 +0000 (02:06 -0700)
Changes to queue in CABAC didn't get propagated to PCM code.

common/cabac.c
common/cabac.h
encoder/cabac.c

index 11988a12fa9d3f151fbfb576f098e8d3fe506a4f..d0888d0959b89482b7055c7d991ccff12aeb59a4 100644 (file)
@@ -768,12 +768,17 @@ void x264_cabac_context_init( x264_cabac_t *cb, int i_slice_type, int i_qp, int
     }
 }
 
-void x264_cabac_encode_init( x264_cabac_t *cb, uint8_t *p_data, uint8_t *p_end )
+void x264_cabac_encode_init_core( x264_cabac_t *cb )
 {
     cb->i_low   = 0;
     cb->i_range = 0x01FE;
     cb->i_queue = -9; // the first bit will be shifted away and not written
     cb->i_bytes_outstanding = 0;
+}
+
+void x264_cabac_encode_init( x264_cabac_t *cb, uint8_t *p_data, uint8_t *p_end )
+{
+    x264_cabac_encode_init_core( cb );
     cb->p_start = p_data;
     cb->p       = p_data;
     cb->p_end   = p_end;
@@ -877,9 +882,9 @@ void x264_cabac_encode_flush( x264_t *h, x264_cabac_t *cb )
     cb->i_queue += 9;
     x264_cabac_putbyte( cb );
     x264_cabac_putbyte( cb );
-    cb->i_low <<= 8 - cb->i_queue;
+    cb->i_low <<= -cb->i_queue;
     cb->i_low |= (0x35a4e4f5 >> (h->i_frame & 31) & 1) << 10;
-    cb->i_queue = 8;
+    cb->i_queue = 0;
     x264_cabac_putbyte( cb );
 
     while( cb->i_bytes_outstanding > 0 )
index 9fc300732ddcf9b3561fa5d5ec8f7c274437d3a9..cec4e51ab3a384bb7a794f63107a4e9ba5dae1fb 100644 (file)
@@ -51,7 +51,7 @@ extern const uint16_t x264_cabac_entropy[128];
 /* init the contexts given i_slice_type, the quantif and the model */
 void x264_cabac_context_init( x264_cabac_t *cb, int i_slice_type, int i_qp, int i_model );
 
-/* encoder only: */
+void x264_cabac_encode_init_core( x264_cabac_t *cb );
 void x264_cabac_encode_init ( x264_cabac_t *cb, uint8_t *p_data, uint8_t *p_end );
 void x264_cabac_encode_decision_c( x264_cabac_t *cb, int i_ctx, int b );
 void x264_cabac_encode_decision_asm( x264_cabac_t *cb, int i_ctx, int b );
index 2df21d4a2f4dc24e22cbfb60a0e0ee5b823964c2..dc18c9d4ecad7bc5c8318aeea20eeb3d11ca914b 100644 (file)
@@ -776,10 +776,7 @@ void x264_macroblock_write_cabac( x264_t *h, x264_cabac_t *cb )
             memcpy( cb->p + i*8, h->mb.pic.p_fenc[2] + i*FENC_STRIDE, 8 );
         cb->p += 64;
 
-        cb->i_low   = 0;
-        cb->i_range = 0x01FE;
-        cb->i_queue = -1;
-        cb->i_bytes_outstanding = 0;
+        x264_cabac_encode_init_core( cb );
 
         h->stat.frame.i_tex_bits += x264_cabac_pos( cb ) - i_mb_pos_tex;
         return;