diff options
| author | Owain Ainsworth <oga@cvs.openbsd.org> | 2010-04-09 22:42:11 +0000 |
|---|---|---|
| committer | Owain Ainsworth <oga@cvs.openbsd.org> | 2010-04-09 22:42:11 +0000 |
| commit | a6231c7677089fb97f52cf7be394e56b109fcc0c (patch) | |
| tree | 82125bc0512ee75f5527f8d8e75731570c3a8f56 | |
| parent | 6f4f728595be47aae68122893a6eb58875a76094 (diff) | |
make more bettah. instead of doing:
switch(type) {
case VREG:
/*something */
break;
case VLNK:
/* something */
break;
default:
panic("wtf?");
}
do_something_that_doesn't_change_type();
switch(type) {
case VREG:
/* nowt */
break;
case VLNK:
n = 0;
break;
default:
panic("wtf?");
}
be a bit less silly and replace the second switch with:
if (type == VLNK)
n = 0;
ok beck@, blambert@
| -rw-r--r-- | sys/nfs/nfs_bio.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/sys/nfs/nfs_bio.c b/sys/nfs/nfs_bio.c index af2103d0b48..c16791483d6 100644 --- a/sys/nfs/nfs_bio.c +++ b/sys/nfs/nfs_bio.c @@ -1,4 +1,4 @@ -/* $OpenBSD: nfs_bio.c,v 1.69 2010/04/09 22:08:04 oga Exp $ */ +/* $OpenBSD: nfs_bio.c,v 1.70 2010/04/09 22:42:10 oga Exp $ */ /* $NetBSD: nfs_bio.c,v 1.25.4.2 1996/07/08 20:47:04 jtc Exp $ */ /* @@ -228,15 +228,10 @@ again: baddr = bp->b_data; error = uiomove(baddr + on, (int)n, uio); } - switch (vp->v_type) { - case VREG: - break; - case VLNK: + + if (vp->v_type == VLNK) n = 0; - break; - default: - panic("nfsbioread: type %x unexpected\n", vp->v_type); - } + if (got_buf) brelse(bp); } while (error == 0 && uio->uio_resid > 0 && n > 0); |