Fix up some sloppy coding practices:

- Delete redundant declarations.
- Add -Wredundant-declarations to Makefile.i386 so they don't come back.
- Delete sloppy COMMON-style declarations of uninitialized data in
  header files.
- Add a few prototypes.
- Clean up warnings resulting from the above.

NB: ioconf.c will still generate a redundant-declaration warning, which
is unavoidable unless somebody volunteers to make `config' smarter.

1 files changed, 4 insertions, 3 deletions

diff --git a/sys/kern/kern_exec.c b/sys/kern/kern_exec.c
index b54e09b042a7..6602f7f2b029 100644
--- a/sys/kern/kern_exec.c
+++ b/sys/kern/kern_exec.c
@@ -28,7 +28,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- *	$Id: kern_exec.c,v 1.2 1994/05/25 09:03:03 rgrimes Exp $
+ *	$Id: kern_exec.c,v 1.3 1994/08/06 09:06:31 davidg Exp $
  */
 
 #include <sys/param.h>
@@ -52,9 +52,10 @@
 
 #include <machine/reg.h>
 
-int exec_extract_strings __P((struct image_params *));
 int *exec_copyout_strings __P((struct image_params *));
 
+static int exec_check_permissions(struct image_params *);
+
 /*
  * execsw_set is constructed for us by the linker.  Each of the items
  * is a pointer to a `const struct execsw', hence the double pointer here.
@@ -479,7 +480,7 @@ exec_copyout_strings(iparams)
  * Check permissions of file to execute.
  *	Return 0 for success or error code on failure.
  */
-int
+static int
 exec_check_permissions(iparams)
 	struct image_params *iparams;
 {