loplugin:externvar

Change-Id: I304144c0ae6b51656ad326a77e9402cc34d5aa18

5 files changed, 24 insertions, 18 deletions

diff --git a/soltools/mkdepend/def.h b/soltools/mkdepend/def.h
index 74445a2ed130..5e96a2b75712 100644
--- a/soltools/mkdepend/def.h
+++ b/soltools/mkdepend/def.h
@@ -191,4 +191,20 @@ void warning1(char *, ...);
 void convert_slashes(char *);
 char *append_slash(char *);
 
+extern char * directives[];
+
+extern struct inclist * inclistp;
+
+extern char * objprefix;
+
+extern char * objsuffix;
+
+extern boolean printed;
+
+extern boolean verbose;
+
+extern boolean show_where_not;
+
+extern boolean warn_multiple;
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/soltools/mkdepend/include.c b/soltools/mkdepend/include.c
index 6712b1a9ce78..e83ac0fbc24b 100644
--- a/soltools/mkdepend/include.c
+++ b/soltools/mkdepend/include.c
@@ -38,12 +38,9 @@ int issymbolic(char * dir, char * component);
 int exists_path(struct IncludesCollection*, char*);
 
 
-extern struct inclist inclist[ MAXFILES ],
-                      *inclistp;
+extern struct inclist inclist[ MAXFILES ];
 extern char *includedirs[ ];
 extern char *notdotdot[ ];
-extern boolean show_where_not;
-extern boolean warn_multiple;
 
 struct inclist *inc_path(char *file, char *include, boolean dot, struct IncludesCollection *incCollection)
 {
diff --git a/soltools/mkdepend/main.c b/soltools/mkdepend/main.c
index c156c19b2019..997ba6dbb87d 100644
--- a/soltools/mkdepend/main.c
+++ b/soltools/mkdepend/main.c
@@ -120,16 +120,16 @@ void redirect(char * makefile);
 struct  inclist inclist[ MAXFILES ],
                 *inclistp = inclist;
 
-struct symhash *maininclist = NULL;
+static struct symhash *maininclist = NULL;
 
 char    *filelist[ MAXFILES ];
 char    *includedirs[ MAXDIRS + 1 ];
 char    *notdotdot[ MAXDIRS ];
 char    *objprefix = "";
 char    *objsuffix = OBJSUFFIX;
-char    *startat = "# DO NOT DELETE";
-int width = 78;
-boolean append = FALSE;
+static char    *startat = "# DO NOT DELETE";
+static int width = 78;
+static boolean append = FALSE;
 boolean printed = FALSE;
 boolean verbose = FALSE;
 boolean show_where_not = FALSE;
@@ -161,7 +161,7 @@ catch (int sig)
 struct sigaction sig_act;
 #endif /* USGISH */
 
-boolean native_win_slashes = FALSE;
+static boolean native_win_slashes = FALSE;
 
 int main(int argc, char    **argv)
 {
diff --git a/soltools/mkdepend/parse.c b/soltools/mkdepend/parse.c
index 01adafbbbd36..250a6a66a702 100644
--- a/soltools/mkdepend/parse.c
+++ b/soltools/mkdepend/parse.c
@@ -38,7 +38,6 @@ int deftype ( char *line, struct filepointer *filep,
 int zero_value(char *exp, struct filepointer *filep,
     struct inclist *file_red, struct symhash *symbols);
 
-extern char *directives[];
 extern struct symhash *maininclist;
 
 int find_includes(struct filepointer *filep, struct inclist *file, struct inclist *file_red, int recursion, boolean failOK, struct IncludesCollection* incCollection, struct symhash *symbols)
@@ -361,7 +360,7 @@ int deftype (char *line, struct filepointer *filep, struct inclist *file_red, st
  * HACK! - so that we do not have to introduce 'symbols' in each cppsetup.c
  * function...  It's safe, functions from cppsetup.c don't return here.
  */
-struct symhash *global_symbols = NULL;
+static struct symhash *global_symbols = NULL;
 
 char * isdefined( char *symbol )
 {
diff --git a/soltools/mkdepend/pr.c b/soltools/mkdepend/pr.c
index 23a93f4b9e9b..fe8d756a636c 100644
--- a/soltools/mkdepend/pr.c
+++ b/soltools/mkdepend/pr.c
@@ -31,14 +31,8 @@ in this Software without prior written authorization from the X Consortium.
 #include <string.h>
 size_t pr( struct inclist *ip, char *file,char *base);
 
-extern struct   inclist inclist[ MAXFILES ],
-                *inclistp;
-extern char *objprefix;
-extern char *objsuffix;
+extern struct   inclist inclist[ MAXFILES ];
 extern int  width;
-extern boolean  printed;
-extern boolean  verbose;
-extern boolean  show_where_not;
 
 void add_include(struct filepointer *filep, struct inclist *file, struct inclist *file_red, char *include, boolean dot, boolean failOK, struct IncludesCollection* incCollection, struct symhash *symbols)
 {