On Thu, Oct 18, 2012 at 11:36:43AM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > On Thu, Oct 18, 2012 at 11:14:47AM +0100, Måns Rullgård wrote: > >> Diego Biurrun <[email protected]> writes: > >> > On Thu, Oct 18, 2012 at 10:50:44AM +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)) > >> >> > > >> >> > I'm mystified why you don't just write "$(2)_DECODER" ... > >> >> > >> >> To allow multiple decoder dependencies of course. > >> > > >> > Where would you need that? You're not making use of the capability > >> > in the patchset ... > >> > >> Maybe one of the remaining 1000 tests does. > > > > What sort of test might need two (or more) decoders? > > When you enjoy a film, do you play the audio or the video first?
Haha :) - patch OK. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
