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 d0536af..6a6708a 100644 (file)
@@ -1,5 +1,10 @@
 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"
        to find_target_beneath.
        * ravenscar-thread.c (ravenscar_prepare_to_store): Pass "ops" to
index 5176bf1..7ba2720 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 304310f..1d77e24 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