[MPlayer-dev-eng] Patch for hi resolution network video

[Ross Finlayson]

At 11:45 AM 7/25/05, Diego Biurrun wrote:
>Ross, could you please review/apply this?

Unfortunately I don't want to install either of these patches, because 
there are problems with both.

The problem with the first patch is that it causes excessive memory 
consumption.  For many users, allocating a 2 MByte buffer will be excessive.

The basic problem here is that JPEG is not a particularly good codec for 
streaming, expecially if your frames get to be as large as 2 MBytes.

The problem with the second patch is that "demux_rtp_codec.cpp" is the 
wrong place to be looking through video frame data to extract width and 
height information.  Instead, this should be done in the JPEG 
decoder.  (And you should probably be doing the same thing for other codecs 
also - e.g., MPEG-4.)


	Ross Finlayson
	LIVE.COM
	<http://www.live.com/>