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

Re: [Xen-devel] [PATCH v2] get_maintainer.pl: Teach brace expansion



Anthony PERARD writes ("[PATCH v2] get_maintainer.pl: Teach brace expansion"):
> It is done by using two different regex, the first one will take care of
> converting ',' inside '{}' to a '|', one by one, as long as there is at
> least two commas. The second regex will do the final convertion of '{,}'
> to '(|)'.

What a palaver.

I don't want to nitpick your Perl code.  I'm basically fine with
anything here so long as it currently gives right answers.

But I wondered if you know about the /x and /e modifiers on perl
regexps.

Anyway,

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

on the assumption that you have tested it.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.