Re: [ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Justin Pettit 

> On Sep 19, 2016, at 8:26 AM, Ben Pfaff  wrote:
> 
> On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit 
> 
> Please drop the changes from "flow-based" to "flow\-based".  The rule
> for "-" versus "\-" in nroff isn't "put \ before every -", but instead
> more like "put \ before a literal - in sample syntax".  The fuller
> version of the rule is that - is a hyphen and \- is a minus sign; you
> want the former in "flow-based" and the latter in e.g. "--dry-run".
> 
> I'll get off my high horse now.
> 
> Anyway, except for those particular changes,
> Acked-by: Ben Pfaff 

Thanks for the explanation.  I made that change and pushed the series to master.

--Justin


___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Ben Pfaff 
On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit 

Please drop the changes from "flow-based" to "flow\-based".  The rule
for "-" versus "\-" in nroff isn't "put \ before every -", but instead
more like "put \ before a literal - in sample syntax".  The fuller
version of the rule is that - is a hyphen and \- is a minus sign; you
want the former in "flow-based" and the latter in e.g. "--dry-run".

I'll get off my high horse now.

Anyway, except for those particular changes,
Acked-by: Ben Pfaff 
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev