diff options
author | Gary Benson <gbenson@redhat.com> | 2015-06-24 16:06:49 +0100 |
---|---|---|
committer | Gary Benson <gbenson@redhat.com> | 2015-06-25 09:31:07 +0100 |
commit | a3be80c3c54e9e2c0d8e9a1ff77c931cd8ebeb60 (patch) | |
tree | 21d6dbc895dd38aebad7bcfdd748dc0284817e15 /gdb | |
parent | 53f7371c77f50aea587cf06bf4cc79273ee1382e (diff) | |
download | gdb-a3be80c3c54e9e2c0d8e9a1ff77c931cd8ebeb60.zip gdb-a3be80c3c54e9e2c0d8e9a1ff77c931cd8ebeb60.tar.gz gdb-a3be80c3c54e9e2c0d8e9a1ff77c931cd8ebeb60.tar.bz2 |
Remove gdb_sysroot NULL checks
Since fed040c6a50399617d8265cbddc7fd21b3f134ef gdb_sysroot is
never NULL. This commit removes all gdb_sysroot NULL checks.
gdb/ChangeLog:
* exec.c (exec_file_locate_attach): Remove gdb_sysroot NULL check.
* infrun.c (follow_exec): Likewise.
* remote.c (remote_filesystem_is_local): Likewise.
* solib.c (solib_find_1): Likewise.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 7 | ||||
-rw-r--r-- | gdb/exec.c | 3 | ||||
-rw-r--r-- | gdb/infrun.c | 2 | ||||
-rw-r--r-- | gdb/remote.c | 3 | ||||
-rw-r--r-- | gdb/solib.c | 29 |
5 files changed, 22 insertions, 22 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a52624b..4ff683d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2015-06-25 Gary Benson <gbenson@redhat.com> + + * exec.c (exec_file_locate_attach): Remove gdb_sysroot NULL check. + * infrun.c (follow_exec): Likewise. + * remote.c (remote_filesystem_is_local): Likewise. + * solib.c (solib_find_1): Likewise. + 2015-06-24 Keith Seitz <keiths@redhat.com> * build-id.c (build_id_to_debug_bfd): Add cleanup to free @@ -154,8 +154,7 @@ exec_file_locate_attach (int pid, int from_tty) /* If gdb_sysroot is not empty and the discovered filename is absolute then prefix the filename with gdb_sysroot. */ - if (gdb_sysroot != NULL && *gdb_sysroot != '\0' - && IS_ABSOLUTE_PATH (exec_file)) + if (*gdb_sysroot != '\0' && IS_ABSOLUTE_PATH (exec_file)) full_exec_path = exec_file_find (exec_file, NULL); if (full_exec_path == NULL) diff --git a/gdb/infrun.c b/gdb/infrun.c index 233d0f8..792f847 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -1133,7 +1133,7 @@ follow_exec (ptid_t ptid, char *execd_pathname) breakpoint_init_inferior (inf_execd); - if (gdb_sysroot != NULL && *gdb_sysroot != '\0') + if (*gdb_sysroot != '\0') { char *name = exec_file_find (execd_pathname, NULL); diff --git a/gdb/remote.c b/gdb/remote.c index f15d75e..68dd99d 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -10538,8 +10538,7 @@ remote_filesystem_is_local (struct target_ops *self) this case we treat the remote filesystem as local if the sysroot is exactly TARGET_SYSROOT_PREFIX and if the stub does not support vFile:open. */ - if (gdb_sysroot != NULL - && strcmp (gdb_sysroot, TARGET_SYSROOT_PREFIX) == 0) + if (strcmp (gdb_sysroot, TARGET_SYSROOT_PREFIX) == 0) { enum packet_support ps = packet_support (PACKET_vFile_open); diff --git a/gdb/solib.c b/gdb/solib.c index 0010c2f..ed1bc25 100644 --- a/gdb/solib.c +++ b/gdb/solib.c @@ -159,23 +159,18 @@ solib_find_1 (char *in_pathname, int *fd, int is_solib) struct cleanup *old_chain = make_cleanup (null_cleanup, NULL); char *sysroot = gdb_sysroot; - if (sysroot != NULL) - { - /* If the absolute prefix starts with "target:" but the - filesystem accessed by the target_fileio_* methods - is the local filesystem then we strip the "target:" - prefix now and work with the local filesystem. This - ensures that the same search algorithm is used for - all local files regardless of whether a "target:" - prefix was used. */ - if (is_target_filename (sysroot) && target_filesystem_is_local ()) - sysroot += strlen (TARGET_SYSROOT_PREFIX); - - if (*sysroot == '\0') - sysroot = NULL; - } - - if (sysroot != NULL) + /* If the absolute prefix starts with "target:" but the filesystem + accessed by the target_fileio_* methods is the local filesystem + then we strip the "target:" prefix now and work with the local + filesystem. This ensures that the same search algorithm is used + for all local files regardless of whether a "target:" prefix was + used. */ + if (is_target_filename (sysroot) && target_filesystem_is_local ()) + sysroot += strlen (TARGET_SYSROOT_PREFIX); + + if (*sysroot == '\0') + sysroot = NULL; + else { int prefix_len = strlen (sysroot); |