gdb/

	Code cleanup.
	* target.c (push_target): Return only void.  Remove the return value
	comment.
	* target.h (push_target): Return only void.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4f49802..b831c4c 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2010-05-23  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       Code cleanup.
+       * target.c (push_target): Return only void.  Remove the return value
+       comment.
+       * target.h (push_target): Return only void.
+
 2010-05-23  Pedro Alves  <pedro@codesourcery.com>
 
        Update gnulib from latest git.

diff --git a/gdb/target.c b/gdb/target.c
index d60ef2a..35a29b7 100644 (file)
--- a/gdb/target.c
+++ b/gdb/target.c
@@ -872,14 +872,11 @@ update_current_target (void)
 /* Push a new target type into the stack of the existing target accessors,
    possibly superseding some of the existing accessors.
 
-   Result is zero if the pushed target ended up on top of the stack,
-   nonzero if at least one target is on top of it.
-
    Rather than allow an empty stack, we always have the dummy target at
    the bottom stratum, so we can call the function vectors without
    checking them.  */
 
-int
+void
 push_target (struct target_ops *t)
 {
   struct target_ops **cur;
@@ -920,9 +917,6 @@ push_target (struct target_ops *t)
   (*cur) = t;
 
   update_current_target ();
-
-  /* Not on top?  */
-  return (t != target_stack);
 }
 
 /* Remove a target_ops vector from the stack, wherever it may be.

diff --git a/gdb/target.h b/gdb/target.h
index d4bd007..6f4b660 100644 (file)
--- a/gdb/target.h
+++ b/gdb/target.h
@@ -1414,7 +1414,7 @@ int target_verify_memory (const gdb_byte *data,
 
 extern void add_target (struct target_ops *);
 
-extern int push_target (struct target_ops *);
+extern void push_target (struct target_ops *);
 
 extern int unpush_target (struct target_ops *);