The additions look good.  

By the way, under what conditions should the date be updated on man pages?

--Justin


On Jul 25, 2011, at 12:18 PM, Ben Pfaff wrote:

> Thanks.  I noticed a few other ways to improve it, so I folded those in,
> ending up with:
> 
> From: Ben Pfaff <[email protected]>
> Date: Mon, 25 Jul 2011 09:30:41 -0700
> Subject: [PATCH] ovs-parse-leaks: Fix typos in manpage.
> 
> ---
> utilities/ovs-parse-leaks.8 |   12 +++++++-----
> 1 files changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/utilities/ovs-parse-leaks.8 b/utilities/ovs-parse-leaks.8
> index 23fef8a..e7bd1c5 100644
> --- a/utilities/ovs-parse-leaks.8
> +++ b/utilities/ovs-parse-leaks.8
> @@ -1,7 +1,7 @@
> .TH ovs\-parse\-leaks 8 "August 2010" "Open vSwitch" "Open vSwitch Manual"
> .
> .SH NAME
> -Bovs\-parse\-leaks \- parses OVS leak checker log files
> +ovs\-parse\-leaks \- parses OVS leak checker log files
> .
> .SH SYNOPSIS
> \fBovs\-parse\-leaks\fR [\fIbinary\fR] \fB< \fIlog\fR
> @@ -22,12 +22,14 @@ argument.  For best results, the binary should have debug 
> symbols.
> .SH OPTIONS
> .TP
> \fB\-\-help\fR
> -Prints a help usage message and exits.
> +Prints a usage message and exits.
> .SH BUGS
> The output can be hard to interpret, especially for a daemon that does
> -not exit in normal operation.
> +not exit in normal operation.  Using \fBovs\-appctl\fR(8) to invoke
> +the \fBexit\fR command that some Open vSwitch daemons support
> +sometimes helps with this.
> .PP
> -\fBovs\-parse\-leaks\fR usually incorrectly reports one or more "bad
> -frees of not-allocated address" errors at the beginning of output.
> +\fBovs\-parse\-leaks\fR usually incorrectly reports one or more ``bad
> +frees of not-allocated address'' errors at the beginning of output.
> These reflect frees of data that were allocated before the leak
> checker was turned on during program initialization.
> -- 
> 1.7.2.5
> 
> 
> 
> On Mon, Jul 25, 2011 at 09:50:05AM -0700, Justin Pettit wrote:
>> Looks good.
>> 
>> --Justin
>> 
>> 
>> On Jul 25, 2011, at 9:27 AM, Ben Pfaff wrote:
>> 
>>> ---
>>> utilities/ovs-parse-leaks.8 |    2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>> 
>>> diff --git a/utilities/ovs-parse-leaks.8 b/utilities/ovs-parse-leaks.8
>>> index 23fef8a..b2393e3 100644
>>> --- a/utilities/ovs-parse-leaks.8
>>> +++ b/utilities/ovs-parse-leaks.8
>>> @@ -1,7 +1,7 @@
>>> .TH ovs\-parse\-leaks 8 "August 2010" "Open vSwitch" "Open vSwitch Manual"
>>> .
>>> .SH NAME
>>> -Bovs\-parse\-leaks \- parses OVS leak checker log files
>>> +ovs\-parse\-leaks \- parses OVS leak checker log files
>>> .
>>> .SH SYNOPSIS
>>> \fBovs\-parse\-leaks\fR [\fIbinary\fR] \fB< \fIlog\fR
>>> -- 
>>> 1.7.4.4
>>> 
>>> _______________________________________________
>>> dev mailing list
>>> [email protected]
>>> http://openvswitch.org/mailman/listinfo/dev
>> 

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to