[ovs-dev] [PATCH 01/14] ovsdb-doc: Distinguish hyphens and minus signs in nroff output.

Ben Pfaff blp at nicira.com
Fri Apr 16 17:34:06 UTC 2010


On Thu, Apr 15, 2010 at 03:36:11PM -0700, Justin Pettit wrote:
> On Apr 8, 2010, at 5:06 PM, Ben Pfaff wrote:
> 
> > -    s = re.sub('([\\\\"\'])', escape, s)
> > +    s = re.sub('([-\\\\"\'])', escape, s)
> 
> It may be useful to add a comment explaining that "\\\\" is being escaped down to "\\" in order to be escaped down "\" for the regular expression library.  It was not obvious to me that this was plan.

Thanks, I added a comment.  I also moved the " and \' before the \\\\ so
that it doesn't look at first glance like there's a \" escape involved.




More information about the dev mailing list