On 08/16/2014 08:20 AM, Florian Mayer wrote: > Hello m4 team, [rearranging your mail a bit]
> > I hope that it is an actual bug this time... Not a bug per se, since, as you point out... > > I am aware of the existence of forloop2, which is listed in the example > code > directory, but I like this function for it's simplicity. ...the manual is trying to point out that the simple version is not robust, and your example is proof of that. However, looking at your proposed patch: existing: > define(`_forloop', `$4'`ifelse($1, `$3', `', `define(`$1', > incr($1))$0($@)')') proposed: > define(`_forloop', `$4'``''`ifelse($1, `$3', `', `define(`$1', > incr($1))$0($@)')') you are merely adding an empty quote to separate $4 from ifelse. Which seems simple enough to do. I'll turn it into a formal patch soon enough. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
