[FFmpeg-cvslog] r16777 - trunk/configure

Diego Biurrun diego
Sun Jan 25 20:42:31 CET 2009


On Sun, Jan 25, 2009 at 07:35:51PM +0000, M?ns Rullg?rd wrote:
> 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.

Very well, dropped then.

Diego




More information about the ffmpeg-cvslog mailing list