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/kern/kern_descrip.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/kern/kern_descrip.c')
| -rw-r--r-- | sys/kern/kern_descrip.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sys/kern/kern_descrip.c b/sys/kern/kern_descrip.c index 893cb85a9eb..1620de45d2c 100644 --- a/sys/kern/kern_descrip.c +++ b/sys/kern/kern_descrip.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_descrip.c,v 1.81 2008/11/25 18:04:38 blambert Exp $ */ +/* $OpenBSD: kern_descrip.c,v 1.82 2009/07/09 22:29:56 thib Exp $ */ /* $NetBSD: kern_descrip.c,v 1.42 1996/03/30 22:24:38 christos Exp $ */ /* @@ -856,10 +856,10 @@ fdinit(struct proc *p) struct filedesc *fdp = p->p_fd; newfdp->fd_fd.fd_cdir = fdp->fd_cdir; - VREF(newfdp->fd_fd.fd_cdir); + vref(newfdp->fd_fd.fd_cdir); newfdp->fd_fd.fd_rdir = fdp->fd_rdir; if (newfdp->fd_fd.fd_rdir) - VREF(newfdp->fd_fd.fd_rdir); + vref(newfdp->fd_fd.fd_rdir); } rw_init(&newfdp->fd_fd.fd_lock, "fdlock"); @@ -902,9 +902,9 @@ fdcopy(struct proc *p) newfdp = pool_get(&fdesc_pool, PR_WAITOK); bcopy(fdp, newfdp, sizeof(struct filedesc)); if (newfdp->fd_cdir) - VREF(newfdp->fd_cdir); + vref(newfdp->fd_cdir); if (newfdp->fd_rdir) - VREF(newfdp->fd_rdir); + vref(newfdp->fd_rdir); newfdp->fd_refcnt = 1; /* |