[XviD-devel] [CVS commit] devapi4 -- ME basing its search on wrong data

[Edouard Gomez]

2003-11-16 15:12:15 GMT patch-100
 
    Summary:
      MEanalysis using wrong mvs + bframe search using wrong mvs
    Revision:
      xvidcore--devapi4--1.0--patch-100
 
    Valgrind reported lot of unitialized reads. These unitialized reads
    helped sysKin finding three bugs:
     - ZeroMacroblock did not reset the cbp field. So for some skipped
       blocks, a test was done on the cbp value...
     - MEanalysis was using wrong mvs from the current bvop (unitialized
       or just wrong in current context). That's because in devapi3,
       bframes
       used to share the same mvs array wheras now, it's one array a
       bvop.
     - Collocated skipped MBs for a bvop didn't reset mvs[0] and b_mvs[0].
 
    modified files:
     src/encoder.c src/motion/estimation_bvop.c src/motion/motion.h
     src/motion/motion_inlines.h src/motion/vop_type_decision.c

-- 
Edouard Gomez