Re: [PATCH] tests defs: sanitize IFS

[Stefano Lattarini]

On Sunday 23 January 2011, Ralf Wildenhues wrote:
> * Stefano Lattarini wrote on Sun, Jan 23, 2011 at 12:56:45PM CET:
> > Done in the attached patch.  OK for maint?
> 
> OK.
> 
> Comments explaining the unobvious bits of why something is done are much
> more helpful than comments explaining the obvious: what is done can be
> seen by reading the code.  Please cf. the respective bits in
> gnulib/tests/init.sh: shorter but actually more informative:
> 
>   # As autoconf-generated configure scripts do, ensure that IFS
>   # is defined initially, so that saving and restoring $IFS works.
>   gl_init_sh_nl_='
> '
>   IFS=" ""    $gl_init_sh_nl_"
> 
True indeed.  I've amended the patch to use that comment, and pushed.

Thanks,
  Stefano