autoconf-patches
[Top][All Lists]
Advanced

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

Re: _AS_PATH_WALK


From: Stepan Kasal
Subject: Re: _AS_PATH_WALK
Date: Wed, 15 Mar 2006 12:47:11 +0100
User-agent: Mutt/1.4.1i

Hello,

> > >   * lib/m4sugar/m4sh.m4 (_AS_PATH_WALK): Do not forget to reset
> > >   IFS even in case of empty `$PATH'.

> > > OK? (even if it makes `configure' scripts somewhat longer still ;-)

well, the following patch would save 170 bytes.

I think setting IFS and unsetting CDPATH is ``sanitizing''.
And I think the stack of depth 1 (as_save_IFS) has no practical value;
using a global default value is IMHO more sincere.

OK to commit?

Have a nice day,
        Stepan

Attachment: autoconf-20060315-ifs.patch
Description: Text document


reply via email to

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