[MPlayer-dev-eng] [PATCH] bframes, dwStart: individual patches

[Oded Shimon]

On Fri, Feb 17, 2006 at 12:53:28AM -0800, Corey Hickey wrote:
> [..]
> @@ -864,6 +866,8 @@
>  	mux_v->bih->biCompression, (char *)&mux_v->bih->biCompression);
>  }
>  
> +    muxer->audio_delay_fix -= sh_video->stream_delay;
> +
>  //if(demuxer->file_format!=DEMUXER_TYPE_AVI) pts_from_bps=0; // it must be 0 for mpeg/asf!
>  
>  // ============= AUDIO ===============
> @@ -958,10 +961,7 @@
>  
>  if (verbose>1) print_wave_header(mux_a->wf);
>  
> -if(audio_delay_fix!=0.0){
> -    mux_a->h.dwStart=audio_delay_fix*mux_a->h.dwRate/mux_a->h.dwScale;
> -    mp_msg(MSGT_MENCODER, MSGL_INFO, MSGTR_SettingAudioDelay,mux_a->h.dwStart*mux_a->h.dwScale/(float)mux_a->h.dwRate);
> -}
> +    muxer->audio_delay_fix += sh_audio->stream_delay;
>  
>  } // if(sh_audio)

Could you explain this?... Why do you add and then remove the stream delay?

BTW, it is interesting that taking an AVI file with a dwstart and 
transcoding it with mencoder to some other format would basically make you 
loose the delay. Personally I would've made the avi dwstart go into the 
mencoder -delay, but mencoder -delay is too far from perfect for that I 
guess...

- ods15