[Ffmpeg-devel] Bug? Subtitles not detected in vob

Michael Niedermayer michaelni
Wed Jan 10 18:37:59 CET 2007


Hi

On Wed, Jan 10, 2007 at 04:44:58PM -0000, Wolfram Gloger wrote:
> Hi,
> 
> > well if you call av_estimate_timings at 80% of a file it would with
> > your change estimate the bitrate and duration based on the last 20% or?
> 
> Yes..
> 
> > now seek back to the start, maybe the seek code ignores duration and
> > start time but thats not guranteed and actually the seek code should
> > use the duration and start time instead of redoing the estimation ...
> 
> Agreed.  I keep getting mixed up between av_find_stream_info and
> av_estimate_timings.  av_estimate_timings really just computes two
> global stream parameters (start_time and duration) and that certainly
> should cover the whole file.
> 
> However, I would argue for av_find_stream_info to be callable anywhere
> within the stream (as it can return significantly different parameters
> eg. in MPEG-TS), and since it calls av_estimate_timings()...

yes, what bad happens if av_find_stream_info() returns local info but
av_estimate_timings() global?


> 
> > > Although this does not affect the issue at hand (just checked), I
> > > would propose this change also because it makes av_estimate_timings
> > > more generic (i.e. it can be called after seeking).
> > 
> > i think iam fine with this change
> 
> ...so here that change without further side-effects.
> 
> Regards,
> Wolfram.
> 
> --- trunk/libavformat/utils.c	Mon Jan  8 16:01:14 2007
> +++ ffmpeg/libavformat/utils.c	Wed Jan 10 17:39:27 2007
> @@ -1561,6 +1561,7 @@
>      int read_size, i, ret;
>      int64_t end_time;
>      int64_t filesize, offset, duration;
> +    offset_t old_offset;
>  
>      /* free previous packet */
>      if (ic->cur_st && ic->cur_st->parser)
> @@ -1580,6 +1581,7 @@
>  
>      /* we read the first packets to get the first PTS (not fully
>         accurate, but it is enough now) */
> +    old_offset = url_fseek(&ic->pb, 0, SEEK_CUR);

change that to url_ftell() and iam happy with the patch

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

No evil is honorable: but death is honorable; therefore death is not evil.
-- Citium Zeno
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070110/30751547/attachment.pgp>



More information about the ffmpeg-devel mailing list