On Fri, Oct 19, 2012 at 10:28:30PM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > On Thu, Oct 18, 2012 at 03:25:44AM +0100, Mans Rullgard wrote: > >> --- a/tests/Makefile > >> +++ b/tests/Makefile > >> @@ -33,6 +33,8 @@ ENCDEC2 = $(call ALLYES, $(firstword $(1))_ENCODER > >> $(lastword $(1))_DECODER \ > >> $(firstword $(2))_ENCODER $(lastword > >> $(2))_DECODER \ > >> $(firstword $(3))_MUXER $(lastword > >> $(3))_DEMUXER) > >> > >> +DEMDEC = $(call ALLYES, $(1)_DEMUXER $(2:%=%_DECODER)) > > > > Now that I'm working on adding dependencies I keep getting confused by > > the differences in argument order between the macros. ENCDEC/ENCDEC2 > > has (de)muxers before en/decoders, for DEMDEC it is the other way > > around. > > > > I suggest changing ENCDEC/ENCDEC2 to have (de)muxers as first arguments > > as well. > > I suggest you suck it up or complain yesterday.
I can still prepare a patch to change it and fix this internal API inconsistency.. We just need to decide which direction to change. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
