[MPlayer-dev-eng] [PATCH] support for external VIDIX

Oded Shimon ods15 at ods15.dyndns.org
Tue Mar 28 21:10:44 CEST 2006


On Tue, Mar 28, 2006 at 08:53:49PM +0200, Nico Sabbi wrote:
> Diego Biurrun wrote:
> > > +if test "$_vidix_internal" = yes || test "$_vidix_external" = yes ; then
> >
> >"test A -o B" is simpler and faster.

And less portable and less readable?....

I recall there being a discussion about this.... IIRC '&&' and '||' won...

BTW, you really don't need to need to care about speed for this.
1. 90% of time of configure is in gcc (proven by making the configure 10 
times faster by using tcc)
2. 'test' is a shell built-in, calling test twice is as expensive as 
making 2 function calls.

- ods15




More information about the MPlayer-dev-eng mailing list