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

[Diego Biurrun]

On Mon, Jul 25, 2005 at 05:51:19PM -0700, Ross Finlayson wrote:
> 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.)

I read that as both patches rejected and will drop them from my patch
tracker then.

Diego