[Mplayer-cvslog] CVS: main/libmpdemux demux_mov.c,1.83,1.84

Gábor Lénárt lgb at lgb.hu
Sat Nov 23 18:06:31 CET 2002


On Sat, Nov 23, 2002 at 06:32:08PM +0100, Michael Niedermayer CVS wrote:
> @@ -527,7 +527,7 @@
>  		break;
>  	    }
>  	    case MOV_FOURCC('s','t','t','s'): {
> -		stream_read_dword(demuxer->stream);
> +		int temp=stream_read_dword(demuxer->stream);
>  		int len=stream_read_dword(demuxer->stream);

Hmmm, I've fixed the bug with moving stream_read_dword()'s after variable
declarations. Imho everyone who can code C *SHOULD* know that you can't do this:

{
	some_function();
	int variable;
....

It's a very stupid bug. Please note, that it's NOT C++, it's C!

- Gábor (larta'H)



More information about the MPlayer-cvslog mailing list