On Wed, Mar 27, 2013 at 03:47:22PM +0200, Martin Storsjö wrote:
> On Wed, 27 Mar 2013, Diego Biurrun wrote:
> >On Wed, Mar 27, 2013 at 11:50:57AM +0200, Martin Storsjö wrote:
> >>On Tue, 26 Mar 2013, Diego Biurrun wrote:
> >>>On Tue, Mar 26, 2013 at 06:26:37PM +0200, Martin Storsjö wrote:
> >>>>This makes standalone compilation of the eamad and eatqi decoders
> >>>>and mpeg1video and mpeg2video encoders succeed. The dependency
> >>>>comes from the shared mpeg12.o file.
> >>>>
> >>>>---
> >>>>Splitting out the pieces from mpeg12.c that is shared for each
> >>>>of these components, which doesn't use ER, isn't all that
> >>>>trivial - patches welcome though.
> >>>
> >>>I have split (most of) the mpeg12 decoder code from the shared code.
> >>>That obviates adding ER to the eamad decoder dependencies.
> >
> >As well as to the mpeg1video and mpeg2video encoders, so only eatqi
> >is left with incomplete dependencies now.
> >
> >I'm unsure if it is preferable to try to separate
> >ff_mpeg1_decode_block_intra() from the rest of the mpeg12 decoder
> >code or just add the ER deps to eatqi.
> 
> I'd go for adding the ER deps to this decoder.

OK, so push your patch, but only the eatqi hunk; the others are obsolete.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to