[MPlayer-dev-eng] [PATCH] Demux problem with Sorenson 3 movie

[Arpi]

Hi,

lol :)
did you know that i've posted this (or the same) patch to -users a week ago?
and it is not commited by reason: it's an ugly hack and may break other movs.

> This is a multi-part message in MIME format.
> 
> 
> Hi
> 
> http://progressive.stream.aol.com/wb/gl/wbonline/progressive/
> thematrix/us/med/animatrixlgfinal_dl.mov
> 
> Do not work with mplayer-0.90rc[34].  It is a bit big, so I did not
> upload it to incoming.  Attached is a patch that 'fixes' it, but
> is prob more of a hack than anything else.  It have audio sync
> problems ....
> 
> Patch was by rizzo, more info at:
> 
>   http://forums.gentoo.org/viewtopic.php?t=34001
>   http://bugs.gentoo.org/show_bug.cgi?id=15180
> 
> 
> Regards,
> 
> -- 
> 
> Martin Schlemmer
> Gentoo Linux Developer, Desktop/System Team Developer
> Cape Town, South Africa
> 
> 
> 
> --- libmpdemux/demux_mov.c	27 Jan 2003 22:29:47 -0000	1.93
> +++ libmpdemux/demux_mov.c	5 Feb 2003 17:58:15 -0000
> @@ -597,7 +597,10 @@
>  		    unsigned int fourcc=stream_read_dword_le(demuxer->stream);
>  		    if(len<8) break; // error
>  		    mp_msg(MSGT_DEMUX,MSGL_V,"MOV: %*s desc #%d: %.4s  (%d bytes)\n"
> ,level,"",i,&fourcc,len-16);
> -		    if(!i){
> +		    if(fourcc!=trak->fourcc && i)
> +			mp_msg(MSGT_DEMUX,MSGL_WARN,MSGTR_MOVvariableFourCC);
> +//		    if(!i)
> +		    {
>  			trak->fourcc=fourcc;
>  			// read type specific (audio/video/time/text etc) header
>  			// NOTE: trak type is not yet known at this point :(((
> @@ -605,8 +608,6 @@
>  			trak->stdata=malloc(trak->stdata_len);
>  			stream_read(demuxer->stream,trak->stdata,trak->stdata_len);
>  		    }
> -		    if(fourcc!=trak->fourcc && i)
> -			mp_msg(MSGT_DEMUX,MSGL_WARN,MSGTR_MOVvariableFourCC);
>  		    if(!stream_seek(demuxer->stream,pos+len)) break;
>  		}
>  		break
>  		break
> 
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 
> 


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino