[MPlayer-dev-eng] Re: [PATCH] 422p rawvideo in plus rawuyvy out

[D Richard Felker III]

On Thu, Jan 29, 2004 at 05:31:43PM +0000, John Earl wrote:
> On Thu, 2004-01-29 at 16:50, D Richard Felker III wrote:
> > On Thu, Jan 29, 2004 at 11:54:14AM +0100, Attila Kinali wrote:
> > > On Wed, 21 Jan 2004 12:00:43 +0000
> > > John Earl <jwe21 at cam.ac.uk> wrote:
> > > 
> > > > > Then please, add more format. We are not going to do 2000 little commits.
> > > > > Take the code in mp_image.h dump the size calculation for ALL know
> > > > > formats and send one patch, once. I'm not willing to review each colorspace
> > > > > addition to rawvideo one by one.
> > > > 
> > > > Done (mplayer-rawvideo, attached).
> > > > 
> > > > Am re-attaching the rawuyvy output patch (useful since there isn't yet
> > > > any 4:2:2 colorspace raw output available).
> > > 
> > > Any comments on this patch ?
> > 
> > Yes, it's nonsense. We don't need 20 different "raw*" ovc's. Instead
> > make one ve_raw that supports any format...
> 
> Had envisaged this being the last one you'd ever need. But you're right,
> it would be better to be rid of both rawrgb and rawyuv in favor of a
> single raw ovc. I am using this functionality so will do this and
> resubmit.

The mailing list sets the Reply-to: header for a reason. Please do NOT
send MPlayer mail off-list!

Rich