[MPlayer-dev-eng] [PATCH] clean up tests in configure

Rich Felker dalias at aerifal.cx
Tue Jul 5 18:41:23 CEST 2005


On Tue, Jul 05, 2005 at 02:42:15AM +0200, Diego Biurrun wrote:
> -a/-o vs &&/||:
> ===============
> 
> You say that
> 
> 1) -a/-o is less readable (even "obfuscated"),
> 2) the possibility for ambiguity with -a/-o is reason enough to reject
>    the patch.
> 
> I disagree on both counts since
> 
> 1a) -a/-o is shorter (and IMO this aids readability on long lines),

then use [ instead of test.

> 1b) -a/-o avoids another test invocation and should thus be faster,

there are no such things as invocations. we're talking
shell-internals.

> 1c) after a short time of getting used to it -a/-o should be equally
>     readable for everyone.

no, it's ugly and ambiguous

> 2) I don't think we run the risk of ever hitting that ambiguity and
>    1a-c) might be a real gain in practise.

essentially zero gain. 90% or more of configure time is spent in gcc
anyway, except on broken archs like windows where fork takes forever.

rich




More information about the MPlayer-dev-eng mailing list