[MPlayer-dev-eng] Re: Why is the mplayer-1.0pre3-dumpvideo.diff patch not applied?

[Tobias Diedrich]

John Earl wrote:

> Sorry, didn't think the patch would be wanted! There was already an 
> online help string (using -of help) but have added a couple lines to the 
> man page in new version, attached.

I think rawvideo would be a better name. :-)
And I think I'm probably not the only one who will find this patch
useful, especially for mpeg encoding scripts.

> +static void write_raw_chunk(FILE *f,int len,void* data){
> +    if(len>0){
> +	if(data){
> +	    // DATA
> +	    fwrite(data,len,1,f);
> +	} else {
> +	    // JUNK
> +	    char *avi_junk_data="[= MPlayer junk data! =]";
> +	    if(len&1) ++len; // padding
> +	    while(len>0){
> +		int l=strlen(avi_junk_data);
> +		if(l>len) l=len;
> +		fwrite(avi_junk_data,l,1,f);
> +		len-=l;
> +	    }
> +	}
> +    }
> +}

You can drop the junk path.

> +    if((unsigned int)len>s->h.dwSuggestedBufferSize) s->h.dwSuggestedBufferSize=len;

This line should be unneeded too.

-- 
Tobias						PGP: http://9ac7e0bc.2ya.com
Be vigilant!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040223/10baff03/attachment.pgp>