Diego Biurrun <[email protected]> writes: > 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.
Why don't you work on something that actually matters instead? -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
