[MPlayer-dev-eng] Bug #1028

[Diego Biurrun]

On Thu, Oct 15, 2009 at 10:19:10PM +0200, Attila Kinali wrote:
> 
> Is Bug #1028[1] still an open issue? I tried the file[2] in question,
> but couldnt see any problems.
> 
> Also, is the patch in the last comment in the report ok and can
> it be commited?
> 
> [1] http://bugzilla.mplayerhq.hu/show_bug.cgi?id=1028
> [2] http://samples.ffmpeg.org/FLV/asian-commercials-are-weird.flv

Here is the patch inline:

--- libmpcodecs/vd_ffmpeg.c.old 2009-05-11 09:43:19.000000000 -0400
+++ libmpcodecs/vd_ffmpeg.c     2009-05-11 09:46:16.000000000 -0400
@@ -248,7 +248,7 @@
     if(vd_use_slices && (lavc_codec->capabilities&CODEC_CAP_DRAW_HORIZ_BAND) && !do_vis_debug)
         ctx->do_slices=1;

-    if(lavc_codec->capabilities&CODEC_CAP_DR1 && !do_vis_debug && lavc_codec->id != CODEC_ID_H264 && lavc_codec->id != CODEC_ID_INTERPLAY_VIDEO && lavc_codec->id != CODEC_ID_ROQ)
+    if(lavc_codec->capabilities&CODEC_CAP_DR1 && !do_vis_debug && lavc_codec->id != CODEC_ID_H264 && lavc_codec->id != CODEC_ID_INTERPLAY_VIDEO && lavc_codec->id != CODEC_ID_ROQ && lavc_codec->id != CODEC_ID_VP6F)
         ctx->do_dr1=1;
     ctx->b_age= ctx->ip_age[0]= ctx->ip_age[1]= 256*256*256*64;
     ctx->ip_count= ctx->b_count= 0;

It sure is a hack, but I'm also sure it will not make things any worse,
so feel free to commit it.

Diego