autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: unoptimized autoconf generated code


From: Eric Blake
Subject: Re: unoptimized autoconf generated code
Date: Tue, 02 Mar 2010 15:42:03 -0700
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666

According to Ralf Wildenhues on 3/2/2010 2:45 PM:
> Hi Eric,
> 
> * Eric Blake wrote on Tue, Mar 02, 2010 at 07:04:52AM CET:
>> Here's an updated version of patch 2/3 of the previous post; I realized
>> that ${a+b} should not be treated as a simple indirection, and added some
>> documentation.  Patch 1/3 and 3/3 are unchanged.
> 
> This patch causes a number of testsuite regressions.  The AS_BOX patch
> has a typo s/@var/@ovar/ in the manual causing a texinfo warning.

Thanks for a review.  Certainly still a work in progress, and I won't
commit without a complete successful testsuite run.

> 
> While working on AS_LITERAL_IF, you might want to fix the typo near the
> beginning of the description:
>   # If EXPRESSION has shell indirections ($var or `expr`), expand
>   # IF-LITERAL, else IF-NOT-LITERAL.
> 
> That should be "has no shell indirections".

Yep.

> 
> If you're really motivated, then you could also look into this code
> snippet taken from the manual:
> 
>   header="sys/some file.h"
>   AS_TR_SH([HAVE_$header])=yes
>   if test "x$HAVE_sys_some_file_h" = xyes; then echo "Have it!"; fi
> 
> which has been broken presumably by one of the earlier AS_LITERAL_IF
> changes already.  (This is not a recent regression, so feel free to
> ignore right now.)

Actually, it's a doc bug.  This is not valid shell variable assignment:

`echo $foo`=yes

So this patch is going in now, while I spend more time fixing the actual
code changes.

-- 
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden

Attachment: autoconf.patch261
Description: Text document

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]