[FFmpeg-cvslog] r16777 - trunk/configure

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> On Sun, Jan 25, 2009 at 06:52:34PM +0000, M?ns Rullg?rd wrote:
>> diego <subversion at mplayerhq.hu> writes:
>> >
>> > Log:
>> > SSE depends on x86.
>> >
>> > --- trunk/configure	Sun Jan 25 18:36:48 2009	(r16776)
>> > +++ trunk/configure	Sun Jan 25 18:52:52 2009	(r16777)
>> > @@ -953,7 +953,7 @@ mmx_deps="x86"
>> >  mmx2_deps="x86 mmx"
>> >  neon_deps="arm"
>> >  ppc4xx_deps="ppc"
>> > -sse_deps="mmx"
>> > +sse_deps="x86 mmx"
>> 
>> mmx already depends on x86 so this isn't necessary.
>
> True, but we already have this "redundancy" in the mmx2 and ssse3 cases
> so I went for consistency.

You could also have removed it from the other ones, for consistency.
When things are inconsistent, either of the variants can be the more
correct.  In this case, having fewer explicit dependencies will
probably make the configure script run slightly faster.

-- 
M?ns Rullg?rd
mans at mansr.com