autoconf-patches
[Top][All Lists]
Advanced

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

Re: FW: [GNU Autoconf 2.65] testsuite: 35 239 271 272 273 274 275 276 27


From: Eric Blake
Subject: Re: FW: [GNU Autoconf 2.65] testsuite: 35 239 271 272 273 274 275 276 277 278 279 280 failed
Date: Tue, 02 Mar 2010 05:55:43 -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/1/2010 11:42 PM:
>>> -[m4_ifvaln([$2], [{ AS_SET_STATUS([$2])])]dnl
>>> -[as_fn_error "_AS_QUOTE([$1])"m4_ifval(AS_MESSAGE_LOG_FD,
>>> -  [ "$LINENO" AS_MESSAGE_LOG_FD])[]m4_ifval([$2], [; }])])
>>> +[as_fn_error m4_default([$2], [$?]) 
>>> "_AS_QUOTE([$1])"m4_ifval(AS_MESSAGE_LOG_FD,
>>> +  [ "$LINENO" AS_MESSAGE_LOG_FD])])
>>
>> This can be simplified, to calling as_fn_error $? "_AS_QUOTE([$1])"...
>>
>> Why?  Because we just called AS_SET_STATUS([$2]) the line before, so $? is
>> guaranteed to be the correct value.  And one less m4_default is that much
>> less m4 effort.
> 
> Erm, the patch just removed that AS_SET_STATUS.  If weighing m4 against
> sh time, I usually try to go for lower sh time.

Phooey - I misread the patch.  Indeed, you are correct: with the removal
of AS_SET_STATUS, your patch is correct as-is, and doesn't need any tweaking.

Hmm, that means that as_fn_error no longer requires that we drag in the
definitions used in AS_SET_STATUS; maybe there's another round of cleanup
patches we can do to separate those two, but it is not as high of a priority.

-- 
Eric Blake   address@hidden    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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