diff options
| author | Thordur I. Bjornsson <thib@cvs.openbsd.org> | 2009-07-09 22:29:57 +0000 |
|---|---|---|
| committer | Thordur I. Bjornsson <thib@cvs.openbsd.org> | 2009-07-09 22:29:57 +0000 |
| commit | 2b856d134632dfdeff9ea4e7dca4d8958fb0f0bb (patch) | |
| tree | 12fb7faf3468b0bb2d1712f8f88a6b5318059de6 /sys/nfs/nfs_vnops.c | |
| parent | 90f3568623813a10b6ae7548eaf13da8a5e5923c (diff) | |
Remove the VREF() macro and replaces all instances with a call to verf(),
which is exactly what the macro does.
Macro's that are nothing more then:
#define FUNCTION(arg) function(arg)
are almost always pointless and should go away.
OK blambert@
Agreed by many.
Diffstat (limited to 'sys/nfs/nfs_vnops.c')
| -rw-r--r-- | sys/nfs/nfs_vnops.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sys/nfs/nfs_vnops.c b/sys/nfs/nfs_vnops.c index ee879632549..aeb0310ee78 100644 --- a/sys/nfs/nfs_vnops.c +++ b/sys/nfs/nfs_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_vnops.c,v 1.115 2009/06/25 15:49:26 thib Exp $ */ +/* $OpenBSD: nfs_vnops.c,v 1.116 2009/07/09 22:29:56 thib Exp $ */ /* $NetBSD: nfs_vnops.c,v 1.62.4.1 1996/07/08 20:26:52 jtc Exp $ */ /* @@ -845,7 +845,7 @@ dorpc: */ if (NFS_CMPFH(np, fhp, fhsize)) { - VREF(dvp); + vref(dvp); newvp = dvp; if (v3) { nfsm_postop_attr(newvp, attrflag); @@ -2438,7 +2438,7 @@ nfs_readdirplusrpc(struct vnode *vp, struct uio *uiop, struct ucred *cred, if (doit) { nfsm_getfh(fhp, fhsize, 1); if (NFS_CMPFH(dnp, fhp, fhsize)) { - VREF(vp); + vref(vp); newvp = vp; np = dnp; } else { @@ -2545,7 +2545,7 @@ nfs_sillyrename(dvp, vp, cnp) sp = malloc(sizeof(struct sillyrename), M_NFSREQ, M_WAITOK); sp->s_cred = crdup(cnp->cn_cred); sp->s_dvp = dvp; - VREF(dvp); + vref(dvp); if (vp->v_type == VDIR) { #ifdef DIAGNOSTIC @@ -2632,7 +2632,7 @@ nfs_lookitup(dvp, name, len, cred, procp, npp) np->n_fhsize = fhlen; newvp = NFSTOV(np); } else if (NFS_CMPFH(dnp, nfhp, fhlen)) { - VREF(dvp); + vref(dvp); newvp = dvp; np = dnp; } else { |