[MPlayer-cvslog] r26807 - trunk/configure

[Aurelien Jacobs]

Diego Biurrun wrote:

> On Sun, May 18, 2008 at 08:34:52PM +0400, Michael Kostylev wrote:
> > 
> > At Sun, 18 May 2008 17:49:32 +0200 (CEST),
> > diego wrote:
> > 
> >  >10l: Add missing parentheses in AltiVec test logic.
> >  >
> >  >      cc_check -maltivec -mabi=altivec \
> >  >          && _altivec_gcc_flags="-maltivec -mabi=altivec" \
> >  > -        || cc_check -faltivec && _altivec_gcc_flags=-faltivec \
> >  > +        || ( cc_check -faltivec && _altivec_gcc_flags=-faltivec ) \
> >  >          || _altivec=no
> > 
> > -        || ( cc_check -faltivec && _altivec_gcc_flags=-faltivec ) \
> > +        || { cc_check -faltivec && _altivec_gcc_flags=-faltivec ; } \
> 
> What would be the advantage?

IIRC ( ) do spawn a subshell, and thus, _altivec_gcc_flags would be
defined only in this subshell.

Aurel