[Ffmpeg-devel] PATCH: H264 YUVJ420P

Michael Niedermayer michaelni
Fri Oct 6 13:59:26 CEST 2006


On Fri, Oct 06, 2006 at 02:43:56AM +0200, elupus wrote:
> > yes everything should be exported, though i would put all the 
> > stuff in a 
> > AVColorspace struct and put a pointer to it in AVCodecContext that way
> > we can add more fields to it later, we can share these 
> > structs and they
> > dont bloat AVCodecContext up by oo much ...
> Here's a new patch with the above handled, I think this is the correct
> method as that will extend easily to all other mpeg formats. It's currently
> only implemented for h264, and the coef's are only output as the index given
> by mpeg. This should extend to encoding too, but that's for another patch.

rejected, there are many ways to implement this but only exporting the fields
raw is not one of them
a implementation must at absolute minimum _contain_ the exact coefficients
somewhere, be it an array of structs into which these indexes or pointers
point or some other construct

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is

More information about the ffmpeg-devel mailing list