From b26a4dcbe3e5f0d370485b579fe895d5059abde2 Mon Sep 17 00:00:00 2001 From: Jan Kratochvil Date: Sun, 23 May 2010 14:23:31 +0000 Subject: [PATCH] gdb/ Code cleanup. * target.c (push_target): Return only void. Remove the return value comment. * target.h (push_target): Return only void. --- gdb/ChangeLog | 7 +++++++ gdb/target.c | 8 +------- gdb/target.h | 2 +- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4f498021318..b831c4ce4fa 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2010-05-23 Jan Kratochvil + + 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 Update gnulib from latest git. diff --git a/gdb/target.c b/gdb/target.c index d60ef2a98fb..35a29b7480a 100644 --- 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 d4bd00759b0..6f4b6609e8c 100644 --- 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 *); -- 2.30.2