From 9b1440374cbe716ac5dab4d3092e67089db36fa5 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 15 Jan 2014 09:30:05 -0700 Subject: [PATCH] pass NULL to TARGET_DEFAULT_RETURN when appropriate This changes instances of TARGET_DEFAULT_RETURN(0) to TARGET_DEFAULT_RETURN(NULL) when appropriate. The use of "0" was a relic from an earlier implementation of make-target-delegates; and I didn't want to go back through the long patch series, fixing up conflicts, just to change this small detail. 2014-02-19 Tom Tromey * target-delegates.c: Rebuild. * target.h (struct target_ops) : Use NULL, not 0, in TARGET_DEFAULT_RETURN. --- gdb/ChangeLog | 8 ++++++++ gdb/target-delegates.c | 14 +++++++------- gdb/target.h | 14 +++++++------- 3 files changed, 22 insertions(+), 14 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0dd90ca8816..7496e3b201b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2014-02-19 Tom Tromey + + * target-delegates.c: Rebuild. + * target.h (struct target_ops) : Use NULL, + not 0, in TARGET_DEFAULT_RETURN. + 2014-02-19 Tom Tromey * target.c (complete_target_initialization): Remove casts. Use diff --git a/gdb/target-delegates.c b/gdb/target-delegates.c index 4585d2592b8..15ab43a12b0 100644 --- a/gdb/target-delegates.c +++ b/gdb/target-delegates.c @@ -609,7 +609,7 @@ delegate_extra_thread_info (struct target_ops *self, struct thread_info *arg1) static char * tdefault_extra_thread_info (struct target_ops *self, struct thread_info *arg1) { - return 0; + return NULL; } static char * @@ -622,7 +622,7 @@ delegate_thread_name (struct target_ops *self, struct thread_info *arg1) static char * tdefault_thread_name (struct target_ops *self, struct thread_info *arg1) { - return 0; + return NULL; } static void @@ -654,7 +654,7 @@ delegate_pid_to_exec_file (struct target_ops *self, int arg1) static char * tdefault_pid_to_exec_file (struct target_ops *self, int arg1) { - return 0; + return NULL; } static void @@ -679,7 +679,7 @@ delegate_get_section_table (struct target_ops *self) static struct target_section_table * tdefault_get_section_table (struct target_ops *self) { - return 0; + return NULL; } static int @@ -772,7 +772,7 @@ delegate_memory_map (struct target_ops *self) static VEC(mem_region_s) * tdefault_memory_map (struct target_ops *self) { - return 0; + return NULL; } static void @@ -811,7 +811,7 @@ delegate_read_description (struct target_ops *self) static const struct target_desc * tdefault_read_description (struct target_ops *self) { - return 0; + return NULL; } static ptid_t @@ -1297,7 +1297,7 @@ delegate_traceframe_info (struct target_ops *self) static struct traceframe_info * tdefault_traceframe_info (struct target_ops *self) { - return 0; + return NULL; } static int diff --git a/gdb/target.h b/gdb/target.h index 34539c16ca5..79805d13afc 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -567,20 +567,20 @@ struct target_ops char *(*to_pid_to_str) (struct target_ops *, ptid_t) TARGET_DEFAULT_FUNC (default_pid_to_str); char *(*to_extra_thread_info) (struct target_ops *, struct thread_info *) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); char *(*to_thread_name) (struct target_ops *, struct thread_info *) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); void (*to_stop) (struct target_ops *, ptid_t) TARGET_DEFAULT_IGNORE (); void (*to_rcmd) (struct target_ops *, char *command, struct ui_file *output) TARGET_DEFAULT_FUNC (default_rcmd); char *(*to_pid_to_exec_file) (struct target_ops *, int pid) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); void (*to_log_command) (struct target_ops *, const char *) TARGET_DEFAULT_IGNORE (); struct target_section_table *(*to_get_section_table) (struct target_ops *) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); enum strata to_stratum; int (*to_has_all_memory) (struct target_ops *); int (*to_has_memory) (struct target_ops *); @@ -672,7 +672,7 @@ struct target_ops change unexpectedly, it should be invalidated, and higher layers will re-fetch it. */ VEC(mem_region_s) *(*to_memory_map) (struct target_ops *) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); /* Erases the region of flash memory starting at ADDRESS, of length LENGTH. @@ -695,7 +695,7 @@ struct target_ops "beneath" target. Returns the description found, or NULL if no description was available. */ const struct target_desc *(*to_read_description) (struct target_ops *ops) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); /* Build the PTID of the thread on which a given task is running, based on LWP and THREAD. These values are extracted from the @@ -987,7 +987,7 @@ struct target_ops cache data; higher layers take care of caching, invalidating, and re-fetching when necessary. */ struct traceframe_info *(*to_traceframe_info) (struct target_ops *) - TARGET_DEFAULT_RETURN (0); + TARGET_DEFAULT_RETURN (NULL); /* Ask the target to use or not to use agent according to USE. Return 1 successful, 0 otherwise. */ -- 2.30.2