Introduced a type `vop_t' for vnode operation functions and used

it 1138 times (:-() in casts and a few more times in declarations.
This change is null for the i386.

The type has to be `typedef int vop_t(void *)' and not `typedef
int vop_t()' because `gcc -Wstrict-prototypes' warns about the
latter.  Since vnode op functions are called with args of different
(struct pointer) types, neither of these function types is any use
for type checking of the arg, so it would be preferable not to use
the complete function type, especially since using the complete
type requires adding 1138 casts to avoid compiler warnings and
another 40+ casts to reverse the function pointer conversions before
calling the functions.

1 files changed, 4 insertions, 2 deletions

diff --git a/sys/fs/msdosfs/denode.h b/sys/fs/msdosfs/denode.h
index b9ea01caa2e0..7b7dc3a8699c 100644
--- a/sys/fs/msdosfs/denode.h
+++ b/sys/fs/msdosfs/denode.h
@@ -1,4 +1,4 @@
-/*	$Id: denode.h,v 1.6 1995/08/02 12:59:47 dfr Exp $ */
+/*	$Id: denode.h,v 1.7 1995/10/29 15:31:43 phk Exp $ */
 /*	$NetBSD: denode.h,v 1.8 1994/08/21 18:43:49 ws Exp $	*/
 
 /*-
@@ -226,10 +226,12 @@ struct defid {
 	/* u_long	defid_gen;	 generation number */
 };
 
-extern int (**msdosfs_vnodeop_p)();
+extern vop_t **msdosfs_vnodeop_p;
+
 int msdosfs_lookup __P((struct vop_lookup_args *));
 int msdosfs_inactive __P((struct vop_inactive_args *));
 int msdosfs_reclaim __P((struct vop_reclaim_args *));
+
 /*
  * Internal service routine prototypes.
  */