On 06/15/2010 11:09 PM, Ralf Wildenhues wrote:
> Undocumented good features are a waste of goodness!  :-)
> 
> I'm not sure in which way this code may overlap, conflict, or profit
> from, Gary's option parsing endeavor, but a code search showed me that
> the macros are used in third-party code already, so might as well get
> these right.  Sorry if there is duplication, this came from 2003.

And ultimately, I'd like to see this moved to the M4sh level (then both
autoconf and autotest get it for free).  But that will be post-2.66.

But since we haven't touched that code in a long time, and people are
using it in the wild, this patch looks fine to me, with one nit fixed first:

> +m4_define_default([AT_first_option_tr],
> +               [m4_bpatsubst(m4_defn([AT_first_option]), -, _)])dnl

Let's use [_] here, in case _ is a macro name.

-- 
Eric Blake   ebl...@redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to