[FFmpeg-devel] [PATCH] FFV1 specification: Merge of FrameHeader01() and GlobalHeader()

[Michael Niedermayer]

On Wed, May 20, 2015 at 09:24:37AM +0200, Jerome Martinez wrote:
> Le 20/05/2015 03:44, Michael Niedermayer a écrit :
> >[...]
> >
> >If you want to allow multiple w/h/colorspace, its probably better
> >to allow multiple global headers as in h264/h265 and put a index
> >in the frame header to choose one of the parameter sets
> >some of the tables are also large IIRC initial_state_delta is.
> >it makes not much sense for this to be anywhere but global
> 
> Proposals for the next version of FFV1 is planned ;-).
> But for the moment, I focus on having a specification of the
> existing versions.
> 
> >maybe none of these headers should have been called frame header
> >i dont know ...
> 

> I thought it would have been better to separates subjects (on one
> side the merge, on another side the renaming).

I dont agree that a A/B -> X change should be factored into
A -> B, B -> X when the interediate state is just wrong
one could have factored it like A->X, B->X

either way, patch merged

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150520/309fad9b/attachment.asc>