[FFmpeg-cvslog] r20938 - trunk/libavcodec/h263.c

Michael Niedermayer michaelni
Fri Jan 1 21:43:35 CET 2010


On Sun, Dec 27, 2009 at 03:32:23PM +0100, diego wrote:
> Author: diego
> Date: Sun Dec 27 15:32:23 2009
> New Revision: 20938
> 
> Log:
> Remove commented-out debug console output.
[...]
> @@ -2642,7 +2636,6 @@ void mpeg4_pred_ac(MpegEncContext * s, D
>  static inline void mpeg4_encode_dc(PutBitContext * s, int level, int n)
>  {
>  #if 1
> -//    if(level<-255 || level>255) printf("dc overflow\n");
>      level+=256;
>      if (n < 4) {
>          /* luminance */
[...]
> @@ -5807,7 +5771,6 @@ no_cplx_est:
>              if(   h_sampling_factor_n==0 || h_sampling_factor_m==0
>                 || v_sampling_factor_n==0 || v_sampling_factor_m==0){
>  
> -//                fprintf(stderr, "illegal scalability header (VERY broken encoder), trying to workaround\n");
>                  s->scalability=0;
>  
>                  *gb= bak;
[...]
>          s->time_base+= time_incr;
>          s->time= s->time_base*s->avctx->time_base.den + time_increment;
>          if(s->workaround_bugs&FF_BUG_UMP4){
>              if(s->time < s->last_non_b_time){
> -//                fprintf(stderr, "header is not mpeg4 compatible, broken encoder, trying to workaround\n");
>                  s->time_base++;
>                  s->time+= s->avctx->time_base.den;
>              }
> @@ -5936,7 +5895,6 @@ static int decode_vop_header(MpegEncCont
>          s->time= (s->last_time_base + time_incr)*s->avctx->time_base.den + time_increment;
>          s->pb_time= s->pp_time - (s->last_non_b_time - s->time);
>          if(s->pp_time <=s->pb_time || s->pp_time <= s->pp_time - s->pb_time || s->pp_time<=0){
> -//            printf("messed up order, maybe after seeking? skipping current b frame\n");
>              return FRAME_SKIPPED;
>          }
>          ff_mpeg4_init_direct_mv(s);

removing above lines makes the code harder to understand


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

> ... defining _GNU_SOURCE...
For the love of all that is holy, and some that is not, don't do that.
-- Luca & Mans
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100101/e3189d82/attachment.pgp>



More information about the ffmpeg-cvslog mailing list