On Fri, Feb 12, 2010 at 4:21 PM, Jason McIntyre <j...@kerhand.co.uk> wrote: > On Fri, Feb 12, 2010 at 02:01:26PM +0000, Owain Ainsworth wrote: >> Index: sys/mprotect.2 >> =================================================================== >> RCS file: /cvs/src/lib/libc/sys/mprotect.2,v >> retrieving revision 1.14 >> diff -u -p -r1.14 mprotect.2 >> --- sys/mprotect.2 31 May 2007 19:19:33 -0000 1.14 >> +++ sys/mprotect.2 12 Feb 2010 14:00:46 -0000 >> @@ -55,7 +55,7 @@ through >> (inclusive). >> If >> .Fa len >> -is 0, then action will be taken on the page that contains >> +is 0, then no action will be taken on the page that contains > > personally i would be tempted to simplify this to: > > If > .Fa len > is 0, no action is taken...
Agreed, I think that this style is much simpler and easy to read. I prefer the change this way too.
