make exec_ops static
authorTom Tromey <tromey@redhat.com>
Fri, 18 Jul 2014 15:27:17 +0000 (09:27 -0600)
committerTom Tromey <tromey@redhat.com>
Fri, 18 Jul 2014 15:59:31 +0000 (09:59 -0600)
While working on some target stack changes, I noticed that exec_ops is
only used from exec.c.  This patch makes it "static".  This is cleaner
and makes it simpler to reason about the use of the target.

Tested by rebuilding.
I'm checking this in as obvious.

2014-07-18  Tom Tromey  <tromey@redhat.com>

* exec.c (exec_ops): Now static.
* exec.h (exec_ops): Don't declare.

gdb/ChangeLog
gdb/exec.c
gdb/exec.h

index d0536af53dc7b44a1ad979d775c2083257569c32..6a6708a7de6b79577075f6bf144f598c0a673c4d 100644 (file)
@@ -1,3 +1,8 @@
+2014-07-18  Tom Tromey  <tromey@redhat.com>
+
+       * exec.c (exec_ops): Now static.
+       * exec.h (exec_ops): Don't declare.
+
 2014-07-18  Tom Tromey  <tromey@redhat.com>
 
        * spu-multiarch.c (spu_region_ok_for_hw_watchpoint): Pass "self"
index 5176bf1ffc0d8e896256ccd3075a232ada19e038..7ba2720b8215a0612fa98d3c258de9507e4f794b 100644 (file)
@@ -61,7 +61,7 @@ void _initialize_exec (void);
 
 /* The target vector for executable files.  */
 
-struct target_ops exec_ops;
+static struct target_ops exec_ops;
 
 /* True if the exec target is pushed on the stack.  */
 static int using_exec_ops;
index 304310fcfc184a94a972fb843a9a085553393435..1d77e24497fad0c28ff1d3640d8ab5c603ae9783 100644 (file)
@@ -29,8 +29,6 @@ struct target_ops;
 struct bfd;
 struct objfile;
 
-extern struct target_ops exec_ops;
-
 #define exec_bfd current_program_space->ebfd
 #define exec_bfd_mtime current_program_space->ebfd_mtime
 #define exec_filename current_program_space->pspace_exec_filename