[MPlayer-dev-eng] [PATCH] Improving handling of PES files

[Nico Sabbi]

Christian Aistleitner wrote:
> Hello,
> 
> currently PES files typically (at least those I recorded via szap and  
> ts2pes) fail to show the correct length of the video stream and have 
> some  issues with computing
> demuxer->priv->has_valid_timestamps. The supplied patch returns the 
> length  as difference between the last and the first pts within the 
> file, if  demuxer->priv-has_valid_timestamps holds. Additionally, the 
> computation of  demuxer->priv->has_valid_timestamps has been seriously 
> reworked. Finally,  due to more accurate length computation, seeking has 
> been improved further  for streams with 
> demuxer->priv->has_valid_timestamps.
> 
> Kind regards,
> Christian
> 
> 

there are two problems with this patch:
1) cosmetics overflow
2) too much seeking (unacceptable, think of what happens on non-file 
streams) and too much complication (that is not worth the result, in 
any case).

If you want precise seeking you should implement -idx instead

-- 
"Without a frontend, mplayer is useless" - someone in mplayer-users