[MPlayer-dev-eng] [PATCH] configure: enhancement for runtime cpu detection and --target support under x86
Rich Felker
dalias at aerifal.cx
Tue Dec 12 17:34:10 CET 2006
On Tue, Dec 12, 2006 at 10:59:25PM +0800, Zuxy Meng wrote:
> >> -elif x86 || x86_64; then
> >> +elif x86 || x86_64 && [ "$_runtime_cpudetection" = no -a -z "$_target"
> >] ; then
> >
> >And do not use -a, use two checks and combine them with && (many more
> >cases of that)
>
> Hmm...[], ! and -a/-o are used elsewhere in the configure. Forgive me
> I really didn't have any idea which alternative provides better
> compatibility.
This is just a stylistic convention of MPlayer. IMO (and others'
opinions too) the syntaxt for test/[ (-a, -o, etc.) is highly
unintuitive to someone reading, while && and || are obvious.
Rich
More information about the MPlayer-dev-eng
mailing list