On Sun, Sep 10, 2017 at 12:45:19PM +0800, tiejun.peng wrote:
> non-interleaved process call avio_seek frequently, it will cause some 
> performance issues especially in network play scene, such as http/ftp 
> protocol.
> 
> Signed-off-by: tiejun.peng <tiejun.p...@foxmail.com>
> ---
>  libavformat/avidec.c | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/avidec.c b/libavformat/avidec.c
> index b8a31dc..0220ea1 100644
> --- a/libavformat/avidec.c
> +++ b/libavformat/avidec.c
> @@ -1520,7 +1520,8 @@ resync:
>          }
>          ast->seek_pos= 0;
>  

> +        /* subtitle stream no need the next process, because subtitle's 
> pkt->dts is always 0 */

The english grammer is not good


> -        if (!avi->non_interleaved && st->nb_index_entries>1 && 
> avi->index_loaded>1) {
> +        if (!avi->non_interleaved && st->nb_index_entries>1 && 
> avi->index_loaded>1 && st->codecpar->codec_type != AVMEDIA_TYPE_SUBTITLE) {

this already checks that there are multiple index entries

Do these all have the same timestamp ? it seems a bit counter intuitiv
to have 2 with dts=0. Please elaborate

also please provide a testcase to reproduce the issue

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

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to