From 2b856d134632dfdeff9ea4e7dca4d8958fb0f0bb Mon Sep 17 00:00:00 2001 From: "Thordur I. Bjornsson" Date: Thu, 9 Jul 2009 22:29:57 +0000 Subject: 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. --- sys/isofs/cd9660/cd9660_lookup.c | 4 ++-- sys/isofs/cd9660/cd9660_vfsops.c | 4 ++-- sys/isofs/udf/udf_vfsops.c | 4 ++-- sys/isofs/udf/udf_vnops.c | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) (limited to 'sys/isofs') diff --git a/sys/isofs/cd9660/cd9660_lookup.c b/sys/isofs/cd9660/cd9660_lookup.c index 6f913280309..208bbab842f 100644 --- a/sys/isofs/cd9660/cd9660_lookup.c +++ b/sys/isofs/cd9660/cd9660_lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cd9660_lookup.c,v 1.15 2007/10/29 13:02:19 chl Exp $ */ +/* $OpenBSD: cd9660_lookup.c,v 1.16 2009/07/09 22:29:55 thib Exp $ */ /* $NetBSD: cd9660_lookup.c,v 1.18 1997/05/08 16:19:59 mycroft Exp $ */ /*- @@ -398,7 +398,7 @@ found: *vpp = tdp; } else if (dp->i_number == dp->i_ino) { brelse(bp); - VREF(vdp); /* we want ourself, ie "." */ + vref(vdp); /* we want ourself, ie "." */ *vpp = vdp; } else { error = cd9660_vget_internal(vdp->v_mount, dp->i_ino, &tdp, diff --git a/sys/isofs/cd9660/cd9660_vfsops.c b/sys/isofs/cd9660/cd9660_vfsops.c index fbf67947c25..62a67c8e750 100644 --- a/sys/isofs/cd9660/cd9660_vfsops.c +++ b/sys/isofs/cd9660/cd9660_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: cd9660_vfsops.c,v 1.49 2009/05/21 23:45:48 krw Exp $ */ +/* $OpenBSD: cd9660_vfsops.c,v 1.50 2009/07/09 22:29:55 thib Exp $ */ /* $NetBSD: cd9660_vfsops.c,v 1.26 1997/06/13 15:38:58 pk Exp $ */ /*- @@ -847,7 +847,7 @@ retry: ip->i_mnt = imp; ip->i_devvp = imp->im_devvp; - VREF(ip->i_devvp); + vref(ip->i_devvp); if (relocated) { /* diff --git a/sys/isofs/udf/udf_vfsops.c b/sys/isofs/udf/udf_vfsops.c index dedd9c61561..85bf2556eb2 100644 --- a/sys/isofs/udf/udf_vfsops.c +++ b/sys/isofs/udf/udf_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: udf_vfsops.c,v 1.28 2009/06/05 04:35:25 krw Exp $ */ +/* $OpenBSD: udf_vfsops.c,v 1.29 2009/07/09 22:29:56 thib Exp $ */ /* * Copyright (c) 2001, 2002 Scott Long @@ -577,7 +577,7 @@ udf_vget(struct mount *mp, ino_t ino, struct vnode **vpp) up->u_dev = ump->um_dev; up->u_ump = ump; vp->v_data = up; - VREF(ump->um_devvp); + vref(ump->um_devvp); lockinit(&up->u_lock, PINOD, "unode", 0, 0); diff --git a/sys/isofs/udf/udf_vnops.c b/sys/isofs/udf/udf_vnops.c index 104e7ae1d93..48b77b38305 100644 --- a/sys/isofs/udf/udf_vnops.c +++ b/sys/isofs/udf/udf_vnops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: udf_vnops.c,v 1.35 2009/06/05 04:35:25 krw Exp $ */ +/* $OpenBSD: udf_vnops.c,v 1.36 2009/07/09 22:29:56 thib Exp $ */ /* * Copyright (c) 2001, 2002 Scott Long @@ -1006,7 +1006,7 @@ udf_lookup(void *v) * If dvp is what's being looked up, then return it. */ if (ap->a_cnp->cn_namelen == 1 && ap->a_cnp->cn_nameptr[0] == '.') { - VREF(dvp); + vref(dvp); *vpp = dvp; return (0); } -- cgit v1.2.3