[ovs-dev] [PATCH] leak-checker: Document "--leak-limit" option

Justin Pettit jpettit at nicira.com
Thu Apr 8 16:55:47 UTC 2010


On Apr 8, 2010, at 9:33 AM, Ben Pfaff wrote:

> The "-"s should be prefixed by \.  I know that we are not consistent
> about this now, but I try not to add any more mistakes.

My first version had it that way, and then I realized the rest of the file didn't do that.  :)  I've now fixed all the "-"s in that file.

> I would add another sentence: "Finding leaks sometimes requires allowing
> the leak file to grow very large, up to 1GB."
> 
> I would add a sentence to the --check-leaks documentation: "Use the
> \fBovs-parse-leaks\fR script to interpret the leak file."


Great suggestions.  I've integrated them and pushed.

--Justin






More information about the dev mailing list