On 04/20/2015 07:16 PM, Nick Bowler wrote: > Turns out the implementation already had a pushdef, and it looks like > the popdef was simply forgotten. Add it now.
Thanks for catching that. > m4_define([m4_set_foreach], > -[m4_pushdef([$2])m4_set_map_sep([$1], [m4_define([$2],], [)$3])]) > +[m4_pushdef([$2])m4_set_map_sep([$1], [m4_define([$2],], [)$3])dnl > +m4_popdef([$2])]) I'm going to push a followup that avoids a 'dnl' in the middle of the macro. With 'dnl' right there, ALL expansions have to further go through one more macro expansion. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
