cvs commit: src/sys/nfsserver nfs_serv.c

Tom Rhodes trhodes at FreeBSD.org
Mon Jan 23 11:28:36 PST 2006


On Mon, 23 Jan 2006 13:56:16 -0500
John Baldwin <jhb at FreeBSD.org> wrote:

> On Saturday 21 January 2006 07:10, Tom Rhodes wrote:
> > trhodes     2006-01-21 12:10:33 UTC
> >
> >   FreeBSD src repository
> >
> >   Modified files:
> >     sys/nfsserver        nfs_serv.c
> >   Log:
> >   Remove some dead code.
> >
> >   Found with:     Coverity Prevent(tm)
> 
> Are you going to revert this change given the replies?

Oh, I didn't interpret the comments as "this is wrong please
back it out."  I just seen replies, both public and private,
complaining about the indentation.  They went like:

stefanf: "Are you sure this is correct?"
  rwatson: "code is a mess in NFS"

ru: quoting the code "bad indentation"
njl quoting the code "bad indentation"

rees (NFSv4 guy): "looks fine to me"

If you, or anyone else for that matter actually wants it
reverted, I'll do that.  I'm not in the mood to argue
with people today, or ever.  :)

-- 
Tom Rhodes


More information about the cvs-all mailing list