On Thu, Sep 09, 2010 at 01:05:53PM +0000, Thordur I Bjornsson wrote: > These two have been pet peeves of mine for some time and I think > they are ripe for the attic. > > VATTR_NULL -> vattr_null; > NULLVP -> NULL; > > Note, I left the regres for diff alone, there no need to change it. > > Comments/OKs? > > Index: share/man/man9/getdevvp.9 > =================================================================== > RCS file: /home/cvs/src/share/man/man9/getdevvp.9,v > retrieving revision 1.7 > diff -u -p -r1.7 getdevvp.9 > --- share/man/man9/getdevvp.9 31 May 2007 19:20:00 -0000 1.7 > +++ share/man/man9/getdevvp.9 9 Sep 2010 11:36:48 -0000 > @@ -80,7 +80,7 @@ and swap areas, among other things. > will create a vnode for a character device and is used in console handling. > .Sh RETURN VALUES > All functions return 0 on success. > -If an error occurs, vpp will point to a NULLVP. > +If an error occurs, vpp will point to a NULL.
This sounds awful. Why not: If an error occurs, vpp will be set to NULL. > See > .Xr getnewvnode 9 > for further return values. -- :wq Claudio
