]> git.sesse.net Git - vlc/commitdiff
Petites corrections de bug dans le motion compensation et le parsage des
authorJean-Marc Dressler <polux@videolan.org>
Sat, 22 Jan 2000 22:48:14 +0000 (22:48 +0000)
committerJean-Marc Dressler <polux@videolan.org>
Sat, 22 Jan 2000 22:48:14 +0000 (22:48 +0000)
blocks non intra.

src/video_parser/vpar_blocks.c
src/video_parser/vpar_motion.c

index ae32f51cd5a744c3eeb14084dc49f3c5f68f4483..8011f28f706e3dc42eef7cdafb25e8f5b8698af7 100644 (file)
@@ -612,7 +612,7 @@ static __inline__ void InitMacroblock( vpar_thread_t * p_vpar,
                         * (2 - p_vpar->picture.b_frame_structure)
                         * p_vpar->sequence.i_chroma_mb_height;
     p_vpar->mb.i_c_x %= p_vpar->sequence.i_chroma_width;
-    if( p_mb->b_motion_field = p_vpar->picture.b_motion_field )
+    if( (p_mb->b_motion_field = p_vpar->picture.b_motion_field) )
     {
         p_mb->i_motion_l_y--;
         p_mb->i_motion_c_y--;
@@ -689,7 +689,6 @@ static __inline__ void MacroblockModes( vpar_thread_t * p_vpar,
         /* Special No-MC macroblock in P pictures (7.6.3.5). */
         memset( p_vpar->slice.pppi_pmv, 0, 8*sizeof(int) );
         memset( p_mb->pppi_motion_vectors, 0, 8*sizeof(int) );
-        
         p_vpar->mb.i_motion_type = MOTION_FRAME;
         p_mb->ppi_field_select[0][0] = ( p_vpar->picture.i_current_structure == BOTTOM_FIELD );
     }
@@ -709,6 +708,7 @@ static __inline__ void MacroblockModes( vpar_thread_t * p_vpar,
                                         [p_vpar->mb.i_motion_type];
     p_vpar->mb.i_mv_format = ppi_mv_format[p_vpar->picture.b_frame_structure]
                                           [p_vpar->mb.i_motion_type];
+    
 
     p_vpar->mb.b_dct_type = 0;
     if( (p_vpar->picture.i_structure == FRAME_STRUCTURE) &&
@@ -870,7 +870,7 @@ if( 0 )
                                     [p_vpar->mb.i_mb_type & MB_INTRA];
     }
     pf_addb = ppf_addb_intra[p_vpar->mb.i_mb_type & MB_INTRA];
-
+            
     /*
      * Effectively decode blocks.
      */
@@ -1178,6 +1178,7 @@ static void vpar_DecodeMPEG2Non( vpar_thread_t * p_vpar, macroblock_t * p_mb, in
     int         i_pos;
     int         i_run;
     int         i_level;
+    int         i_quant_type;
     boolean_t   b_sign;
     int *       ppi_quant[2];
     
@@ -1188,10 +1189,11 @@ static void vpar_DecodeMPEG2Non( vpar_thread_t * p_vpar, macroblock_t * p_mb, in
 
     /* Determine whether it is luminance or not (chrominance) */
     i_type = ( i_cc + 1 ) >> 1;
-
+    i_quant_type = (!i_type) || (p_vpar->sequence.i_chroma_format == CHROMA_420);
+    
     /* Give a pointer to the quantization matrices for intra blocks */
-    ppi_quant[0] = p_vpar->sequence.nonintra_quant.pi_matrix;
-    ppi_quant[1] = p_vpar->sequence.chroma_nonintra_quant.pi_matrix;
+    ppi_quant[1] = p_vpar->sequence.nonintra_quant.pi_matrix;
+    ppi_quant[0] = p_vpar->sequence.chroma_nonintra_quant.pi_matrix;
 
     /* Decoding of the AC coefficients */
     
@@ -1242,7 +1244,7 @@ static void vpar_DecodeMPEG2Non( vpar_thread_t * p_vpar, macroblock_t * p_mb, in
             case DCT_EOB:
                 if( i_nc <= 0 )
                 {
-                    p_mb->pf_idct[i_b] = vdec_SparseIDCT;
+                    p_mb->pf_idct[i_b] = vdec_IDCT;//vdec_SparseIDCT;
                     p_mb->pi_sparse_pos[i_b] = i_coef;
                 }
                 else
@@ -1265,9 +1267,8 @@ if( i_parse >= 64 )
 }
         
         i_pos = pi_scan[p_vpar->picture.b_alternate_scan][i_parse];
-        i_level = ( i_level *
-                    p_vpar->slice.i_quantizer_scale *
-                    ppi_quant[i_type][i_pos] ) >> 5;
+        i_level = ( ((i_level << 1) + 1) * p_vpar->slice.i_quantizer_scale 
+                    * ppi_quant[i_quant_type][i_pos] ) >> 5;
         p_mb->ppi_blocks[i_b][i_pos] = b_sign ? -i_level : i_level;
     }
 fprintf( stderr, "Non intra MPEG2 end (%d)\n", i_b );
@@ -1302,11 +1303,11 @@ static void vpar_DecodeMPEG2Intra( vpar_thread_t * p_vpar, macroblock_t * p_mb,
 
     /* Determine whether it is luminance or not (chrominance) */
     i_type = ( i_cc + 1 ) >> 1;
-    i_quant_type = i_type | (p_vpar->sequence.i_chroma_format == CHROMA_420);
+    i_quant_type = (!i_type) | (p_vpar->sequence.i_chroma_format == CHROMA_420);
 
     /* Give a pointer to the quantization matrices for intra blocks */
-    ppi_quant[0] = p_vpar->sequence.intra_quant.pi_matrix;
-    ppi_quant[1] = p_vpar->sequence.chroma_intra_quant.pi_matrix;
+    ppi_quant[1] = p_vpar->sequence.intra_quant.pi_matrix;
+    ppi_quant[0] = p_vpar->sequence.chroma_intra_quant.pi_matrix;
 
 #if 0    
     /* Decoding of the DC intra coefficient */
@@ -1477,7 +1478,7 @@ static void vpar_DecodeMPEG2Intra( vpar_thread_t * p_vpar, macroblock_t * p_mb,
             case DCT_EOB:
                 if( i_nc <= 1 )
                 {
-                    p_mb->pf_idct[i_b] = vdec_SparseIDCT;
+                    p_mb->pf_idct[i_b] = vdec_IDCT;//vdec_SparseIDCT;
                     p_mb->pi_sparse_pos[i_b] = i_coef;
                 }
                 else
index 7ccfd07cf8b0fab028d330095de9b61982897e20..5bf6bbc65280f69abb6eb4c09b2596e688870120 100644 (file)
@@ -206,6 +206,8 @@ void vpar_MPEG2MotionVector( vpar_thread_t * p_vpar, macroblock_t * p_mb, int i_
         vpar_MotionVector( p_vpar, p_mb, 0, i_s, 0 );
         p_vpar->slice.pppi_pmv[1][i_s][0] = p_vpar->slice.pppi_pmv[0][i_s][0];
         p_vpar->slice.pppi_pmv[1][i_s][1] = p_vpar->slice.pppi_pmv[0][i_s][1];
+        p_mb->pppi_motion_vectors[1][i_s][0] = p_vpar->slice.pppi_pmv[0][i_s][0];
+        p_mb->pppi_motion_vectors[1][i_s][1] = p_vpar->slice.pppi_pmv[0][i_s][1];
     }
     else
     {
@@ -216,219 +218,4 @@ void vpar_MPEG2MotionVector( vpar_thread_t * p_vpar, macroblock_t * p_mb, int i_
     }
 }
 
-#if 0
 
-/*****************************************************************************
- * vpar_motion.c : motion vectors parsing
- * (c)1999 VideoLAN
- *****************************************************************************/
-
-/*****************************************************************************
- * Preamble
- *****************************************************************************/
-#include <errno.h>
-#include <stdlib.h>
-#include <stdio.h>
-#include <unistd.h>
-#include <string.h>
-#include <sys/uio.h>
-#include <X11/Xlib.h>
-#include <X11/extensions/XShm.h>
-
-#include "config.h"
-#include "common.h"
-#include "mtime.h"
-#include "vlc_thread.h"
-
-#include "intf_msg.h"
-#include "debug.h"                    /* ?? temporaire, requis par netlist.h */
-
-#include "input.h"
-#include "input_netlist.h"
-#include "decoder_fifo.h"
-#include "video.h"
-#include "video_output.h"
-
-#include "vdec_idct.h"
-#include "video_decoder.h"
-#include "vdec_motion.h"
-
-#include "vpar_blocks.h"
-#include "vpar_headers.h"
-#include "video_fifo.h"
-#include "vpar_synchro.h"
-#include "video_parser.h"
-#include "vpar_motion.h"
-
-
-
-/*
- * Local prototypes
- */
-
-/****************************************************************************
- * vpar_MotionCode : Parse the next motion code
- ****************************************************************************/
-static __inline__ int vpar_MotionCode( vpar_thread_t * p_vpar )
-{
-    int i_code;
-    static lookup_t pl_mv_tab0[8] = 
-        { {-1,0}, {3,3}, {2,2}, {2,2}, {1,1}, {1,1}, {1,1}, {1,1} };
-    /* Table B-10, motion_code, codes 0000011 ... 000011x */
-    static lookup_t pl_mv_tab1[8] =
-        { {-1,0}, {-1,0}, {-1,0}, {7,6}, {6,6}, {5,6}, {4,5}, {4,5} };
-    /* Table B-10, motion_code, codes 0000001100 ... 000001011x */
-    static lookup_t pl_mv_tab2[12] = {
-        {16,9}, {15,9}, {14,9}, {13,9},
-        {12,9}, {11,9}, {10,8}, {10,8},
-        {9,8},  {9,8},  {8,8},  {8,8} };
-    
-    if( GetBits(&p_vpar->bit_stream, 1) )
-    {
-        return 0;
-    }
-    if( (i_code = ShowBits(&p_vpar->bit_stream, 9)) >= 64 )
-    {
-        i_code >>= 6;
-        RemoveBits( &p_vpar->bit_stream, pl_mv_tab0[i_code].i_length );
-        return( GetBits(&p_vpar->bit_stream, 1) ? -pl_mv_tab0[i_code].i_value : pl_mv_tab0[i_code].i_value );
-    }
-
-    if( i_code >= 24 )
-    {
-        i_code >>= 3;
-        RemoveBits( &p_vpar->bit_stream, pl_mv_tab1[i_code].i_length );
-        return( GetBits(&p_vpar->bit_stream, 1) ? -pl_mv_tab1[i_code].i_value : pl_mv_tab1[i_code].i_value );
-    }
-
-    if( (i_code -= 12) < 0 )
-    {
-        p_vpar->picture.b_error = 1;
-        intf_DbgMsg( "vpar debug: Invalid motion_vector code\n" );
-        return 0;
-    }
-
-    RemoveBits( &p_vpar->bit_stream, pl_mv_tab2[i_code].i_length );
-    return( GetBits(&p_vpar->bit_stream, 1) ? -pl_mv_tab2[i_code].i_value : pl_mv_tab2[i_code].i_value );            
-}
-
-/****************************************************************************
- * vpar_DecodeMotionVector : decode a motion_vector
- ****************************************************************************/
-static __inline__ void vpar_DecodeMotionVector( int * pi_prediction, int i_r_size,
-        int i_motion_code, int i_motion_residual, int i_full_pel )
-{
-    int i_limit, i_vector;
-
-    /* ISO/IEC 13818-1 section 7.6.3.1 */
-    i_limit = 16 << i_r_size;
-    i_vector = *pi_prediction >> i_full_pel;
-
-    if( i_motion_code > 0 )
-    {
-        i_vector += ((i_motion_code-1) << i_r_size) + i_motion_residual + 1;
-        if( i_vector >= i_limit )
-            i_vector -= i_limit + i_limit;
-    }
-    else if( i_motion_code < 0 )
-    {
-        i_vector -= ((-i_motion_code-1) << i_r_size) + i_motion_residual + 1;
-        if( i_vector < -i_limit )
-            i_vector += i_limit + i_limit;
-    }
-
-    *pi_prediction = i_vector << i_full_pel;
-}
-
-/****************************************************************************
- * vpar_MotionVector : Parse the next motion_vector field
- ****************************************************************************/
-void vpar_MotionVector( vpar_thread_t * p_vpar, macroblock_t * p_mb, int i_r,
-        int i_s, int i_full_pel )
-{
-    int i_motion_code, i_motion_residual;
-    int i_r_size;
-
-    i_r_size = p_vpar->picture.ppi_f_code[i_s][0]-1;
-    i_motion_code = vpar_MotionCode( p_vpar );
-    i_motion_residual = (i_r_size != 0 && i_motion_code != 0) ?
-                        GetBits( &p_vpar->bit_stream, i_r_size) : 0;
-    vpar_DecodeMotionVector( &p_vpar->slice.pppi_pmv[i_r][i_s][0], i_r_size,
-                             i_motion_code, i_motion_residual, i_full_pel );
-    p_mb->pppi_motion_vectors[i_r][i_s][0] = p_vpar->slice.pppi_pmv[i_r][i_s][0];
-
-    if( p_vpar->mb.b_dmv )
-    {
-        if( GetBits(&p_vpar->bit_stream, 1) )
-        {
-            p_mb->pi_dm_vector[0] = GetBits( &p_vpar->bit_stream, 1 ) ? -1 : 1;
-        }
-        else
-        {
-            p_mb->pi_dm_vector[0] = 0;
-        }
-    }
-    
-    i_r_size = p_vpar->picture.ppi_f_code[i_s][1]-1;
-    i_motion_code = vpar_MotionCode( p_vpar );
-    i_motion_residual = (i_r_size != 0 && i_motion_code != 0) ?
-                        GetBits( &p_vpar->bit_stream, i_r_size) : 0;
-
-    if( (p_vpar->mb.i_mv_format == MOTION_FIELD) && (p_vpar->picture.b_frame_structure) )
-         p_vpar->slice.pppi_pmv[i_r][i_s][1] >>= 1;   
-    vpar_DecodeMotionVector( &p_vpar->slice.pppi_pmv[i_r][i_s][1], i_r_size,
-                             i_motion_code, i_motion_residual, i_full_pel );
-    
-    if( (p_vpar->mb.i_mv_format == MOTION_FIELD) && (p_vpar->picture.b_frame_structure) )
-         p_vpar->slice.pppi_pmv[i_r][i_s][1] <<= 1;   
-    p_mb->pppi_motion_vectors[i_r][i_s][1] = p_vpar->slice.pppi_pmv[i_r][i_s][1];
-
-    if( p_vpar->mb.b_dmv )
-    {
-        if( GetBits(&p_vpar->bit_stream, 1) )
-        {
-            p_mb->pi_dm_vector[1] = GetBits( &p_vpar->bit_stream, 1 ) ? -1 : 1;
-        }
-        else
-        {
-            p_mb->pi_dm_vector[1] = 0;
-        }
-    }
-}
-
-/*****************************************************************************
- * vpar_MPEG1MotionVector : Parse the next MPEG-1 motion vectors
- *****************************************************************************/
-void vpar_MPEG1MotionVector( vpar_thread_t * p_vpar, macroblock_t * p_mb, int i_s )
-{
-    vpar_MotionVector( p_vpar, p_mb, 0, i_s, p_vpar->picture.pb_full_pel_vector[i_s] );
-}
-
-/*****************************************************************************
- * vpar_MPEG2MotionVector : Parse the next MPEG-2 motion_vectors field
- *****************************************************************************/
-void vpar_MPEG2MotionVector( vpar_thread_t * p_vpar, macroblock_t * p_mb, int i_s )
-{
-    if( p_vpar->mb.i_mv_count == 1 )
-    {
-        if( p_vpar->mb.i_mv_format == MOTION_FIELD && !p_vpar->mb.b_dmv )
-        {
-            p_mb->ppi_field_select[0][i_s] = p_mb->ppi_field_select[1][i_s]
-                                            = GetBits( &p_vpar->bit_stream, 1 );
-        }
-        vpar_MotionVector( p_vpar, p_mb, 0, i_s, 0 );
-        p_vpar->slice.pppi_pmv[1][i_s][0] = p_vpar->slice.pppi_pmv[0][i_s][0];
-        p_vpar->slice.pppi_pmv[1][i_s][1] = p_vpar->slice.pppi_pmv[0][i_s][1];
-    }
-    else
-    {
-        p_mb->ppi_field_select[0][i_s] = GetBits( &p_vpar->bit_stream, 1 );
-        vpar_MotionVector( p_vpar, p_mb, 0, i_s, 0 );
-        p_mb->ppi_field_select[1][i_s] = GetBits( &p_vpar->bit_stream, 1 );
-        vpar_MotionVector( p_vpar, p_mb, 1, i_s, 0 );
-    }
-}
-
-#endif