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?

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to