Stylistic overhaul of vnops tables.

        1. Remove comment stating the blatantly obvious.
        2. Align in two columns.
        3. Sort all but the default element alphabetically.
        4. Remove XXX comments pointing out entries not needed.

1 files changed, 43 insertions, 49 deletions

diff --git a/sys/ufs/mfs/mfs_vnops.c b/sys/ufs/mfs/mfs_vnops.c
index 10e57ee..71c799f 100644
--- a/sys/ufs/mfs/mfs_vnops.c
+++ b/sys/ufs/mfs/mfs_vnops.c
@@ -31,7 +31,7 @@
  * SUCH DAMAGE.
  *
  *	@(#)mfs_vnops.c	8.11 (Berkeley) 5/22/95
- * $Id: mfs_vnops.c,v 1.20 1997/09/21 22:10:02 gibbs Exp $
+ * $Id: mfs_vnops.c,v 1.21 1997/10/10 18:17:42 phk Exp $
  */
 
 #include <sys/param.h>
@@ -65,54 +65,48 @@ static int	mfs_strategy __P((struct vop_strategy_args *)); /* XXX */
  */
 vop_t **mfs_vnodeop_p;
 static struct vnodeopv_entry_desc mfs_vnodeop_entries[] = {
-	{ &vop_default_desc, (vop_t *)vn_default_error },
-	{ &vop_lookup_desc, (vop_t *)mfs_lookup },	/* lookup */
-/* XXX: vop_cachedlookup */
-	{ &vop_create_desc, (vop_t *)mfs_create },	/* create */
-/* XXX: vop_whiteout */
-	{ &vop_mknod_desc, (vop_t *)mfs_mknod },	/* mknod */
-	{ &vop_open_desc, (vop_t *)mfs_open },		/* open */
-	{ &vop_close_desc, (vop_t *)mfs_close },	/* close */
-	{ &vop_access_desc, (vop_t *)mfs_access },	/* access */
-	{ &vop_getattr_desc, (vop_t *)mfs_getattr },	/* getattr */
-	{ &vop_setattr_desc, (vop_t *)mfs_setattr },	/* setattr */
-	{ &vop_read_desc, (vop_t *)mfs_read },		/* read */
-	{ &vop_write_desc, (vop_t *)mfs_write },	/* write */
-/* XXX: vop_lease */
-	{ &vop_ioctl_desc, (vop_t *)mfs_ioctl },	/* ioctl */
-	{ &vop_poll_desc, (vop_t *)mfs_poll },		/* poll */
-	{ &vop_revoke_desc, (vop_t *)mfs_revoke },	/* revoke */
-	{ &vop_mmap_desc, (vop_t *)mfs_mmap },		/* mmap */
-	{ &vop_fsync_desc, (vop_t *)spec_fsync },	/* fsync */
-	{ &vop_seek_desc, (vop_t *)mfs_seek },		/* seek */
-	{ &vop_remove_desc, (vop_t *)mfs_remove },	/* remove */
-	{ &vop_link_desc, (vop_t *)mfs_link },		/* link */
-	{ &vop_rename_desc, (vop_t *)mfs_rename },	/* rename */
-	{ &vop_mkdir_desc, (vop_t *)mfs_mkdir },	/* mkdir */
-	{ &vop_rmdir_desc, (vop_t *)mfs_rmdir },	/* rmdir */
-	{ &vop_symlink_desc, (vop_t *)mfs_symlink },	/* symlink */
-	{ &vop_readdir_desc, (vop_t *)mfs_readdir },	/* readdir */
-	{ &vop_readlink_desc, (vop_t *)mfs_readlink },	/* readlink */
-	{ &vop_abortop_desc, (vop_t *)mfs_abortop },	/* abortop */
-	{ &vop_inactive_desc, (vop_t *)mfs_inactive },	/* inactive */
-	{ &vop_reclaim_desc, (vop_t *)ufs_reclaim },	/* reclaim */
-	{ &vop_lock_desc, (vop_t *)mfs_lock },		/* lock */
-	{ &vop_unlock_desc, (vop_t *)mfs_unlock },	/* unlock */
-	{ &vop_bmap_desc, (vop_t *)mfs_bmap },		/* bmap */
-	{ &vop_strategy_desc, (vop_t *)mfs_strategy },	/* strategy */
-	{ &vop_print_desc, (vop_t *)mfs_print },	/* print */
-	{ &vop_islocked_desc, (vop_t *)mfs_islocked },	/* islocked */
-	{ &vop_pathconf_desc, (vop_t *)mfs_pathconf },	/* pathconf */
-	{ &vop_advlock_desc, (vop_t *)mfs_advlock },	/* advlock */
-	{ &vop_blkatoff_desc, (vop_t *)mfs_blkatoff },	/* blkatoff */
-	{ &vop_valloc_desc, (vop_t *)mfs_valloc },	/* valloc */
-/* XXX: vop_reallocblks */
-	{ &vop_vfree_desc, (vop_t *)mfs_vfree },	/* vfree */
-	{ &vop_truncate_desc, (vop_t *)mfs_truncate },	/* truncate */
-	{ &vop_update_desc, (vop_t *)mfs_update },	/* update */
-/* XXX: vop_getpages */
-/* XXX: vop_putpages */
-	{ &vop_bwrite_desc, (vop_t *)mfs_bwrite },	/* bwrite */
+	{ &vop_default_desc,		(vop_t *) vn_default_error },
+	{ &vop_abortop_desc,		(vop_t *) mfs_abortop },
+	{ &vop_access_desc,		(vop_t *) mfs_access },
+	{ &vop_advlock_desc,		(vop_t *) mfs_advlock },
+	{ &vop_blkatoff_desc,	(vop_t *) mfs_blkatoff },
+	{ &vop_bmap_desc,		(vop_t *) mfs_bmap },
+	{ &vop_bwrite_desc,		(vop_t *) mfs_bwrite },
+	{ &vop_close_desc,		(vop_t *) mfs_close },
+	{ &vop_create_desc,		(vop_t *) mfs_create },
+	{ &vop_fsync_desc,		(vop_t *) spec_fsync },
+	{ &vop_getattr_desc,		(vop_t *) mfs_getattr },
+	{ &vop_inactive_desc,	(vop_t *) mfs_inactive },
+	{ &vop_ioctl_desc,		(vop_t *) mfs_ioctl },
+	{ &vop_islocked_desc,	(vop_t *) mfs_islocked },
+	{ &vop_link_desc,		(vop_t *) mfs_link },
+	{ &vop_lock_desc,		(vop_t *) mfs_lock },
+	{ &vop_lookup_desc,		(vop_t *) mfs_lookup },
+	{ &vop_mkdir_desc,		(vop_t *) mfs_mkdir },
+	{ &vop_mknod_desc,		(vop_t *) mfs_mknod },
+	{ &vop_mmap_desc,		(vop_t *) mfs_mmap },
+	{ &vop_open_desc,		(vop_t *) mfs_open },
+	{ &vop_pathconf_desc,	(vop_t *) mfs_pathconf },
+	{ &vop_poll_desc,		(vop_t *) mfs_poll },
+	{ &vop_print_desc,		(vop_t *) mfs_print },
+	{ &vop_read_desc,		(vop_t *) mfs_read },
+	{ &vop_readdir_desc,		(vop_t *) mfs_readdir },
+	{ &vop_readlink_desc,	(vop_t *) mfs_readlink },
+	{ &vop_reclaim_desc,		(vop_t *) ufs_reclaim },
+	{ &vop_remove_desc,		(vop_t *) mfs_remove },
+	{ &vop_rename_desc,		(vop_t *) mfs_rename },
+	{ &vop_revoke_desc,		(vop_t *) mfs_revoke },
+	{ &vop_rmdir_desc,		(vop_t *) mfs_rmdir },
+	{ &vop_seek_desc,		(vop_t *) mfs_seek },
+	{ &vop_setattr_desc,		(vop_t *) mfs_setattr },
+	{ &vop_strategy_desc,	(vop_t *) mfs_strategy },
+	{ &vop_symlink_desc,		(vop_t *) mfs_symlink },
+	{ &vop_truncate_desc,	(vop_t *) mfs_truncate },
+	{ &vop_unlock_desc,		(vop_t *) mfs_unlock },
+	{ &vop_update_desc,		(vop_t *) mfs_update },
+	{ &vop_valloc_desc,		(vop_t *) mfs_valloc },
+	{ &vop_vfree_desc,		(vop_t *) mfs_vfree },
+	{ &vop_write_desc,		(vop_t *) mfs_write },
 	{ NULL, NULL }
 };
 static struct vnodeopv_desc mfs_vnodeop_opv_desc =