[Ffmpeg-cvslog] CVS: ffmpeg/libavformat sierravmd.c,1.14,1.15

[Kostya]

On Sat, Mar 11, 2006 at 12:42:19PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Sat, Mar 11, 2006 at 05:28:00AM +0100, Konstantin Shishkov CVS wrote:
> [...]
> >          pkt->pts = vmd->audio_sample_counter;
> >          pkt->pts *= 90000;
> >          pkt->pts /= vmd->sample_rate;
> > -//        pkt->pts /= vmd->audio_frame_divisor;
> > +        pkt->pts /= vmd->audio_channels;
> >          vmd->audio_sample_counter += vmd_calculate_audio_duration(
> >              pkt->data, pkt->size, vmd->audio_block_align);
> 
> this should be
> 
> assert(vmd->audio_sample_counter % vmd->audio_block_align == 0);
> pkt->pts = vmd->audio_sample_counter / vmd->audio_block_align;
> vmd->audio_sample_counter += vmd_calculate_audio_duration(pkt->data, pkt->size, vmd->audio_block_align);
> 
> and av_set_pts_info(st, 64, block_align, sample_rate * channels);
> or something similar
> 
> [...]
> -- 
> Michael
> 
> 
I agree, yet this is not final variant (some VMDs are still not synced) and I will play with pts.