diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2017-04-05 15:15:59 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2017-04-05 15:15:59 -0400 |
commit | 4e9868d4e0c8e45505876901d22c021dd36972a8 (patch) | |
tree | 279804eb170424ac97824ec1540120a0763cbfda /gdb/common | |
parent | 9bf2a700667c53003ece783c05e8b355801105f2 (diff) | |
download | gdb-4e9868d4e0c8e45505876901d22c021dd36972a8.zip gdb-4e9868d4e0c8e45505876901d22c021dd36972a8.tar.gz gdb-4e9868d4e0c8e45505876901d22c021dd36972a8.tar.bz2 |
ptid_{lwp,tid}_p: Remove unnecessary checks
The calls to ptid_equal in ptid_lwp_p and ptid_tid_p that compare the
argument to minus_one_ptid and null_ptid are not necessary. The calls
in question are:
if (ptid_equal (minus_one_ptid, ptid)
|| ptid_equal (null_ptid, ptid))
return 0;
minus_one_ptid is { .pid = -1, .lwp = 0, .tid = 0 }
null_ptid is { .pid = 0, .lwp = 0, .tid = 0 }
If the ptid argument is either of them, the statements
return (ptid_get_lwp (ptid) != 0);
and
return (ptid_get_tid (ptid) != 0);
will yield the same result (0/false).
gdb/ChangeLog:
* common/ptid.c (ptid_lwp_p, ptid_tid_p): Remove comparison with
minus_one_ptid and null_ptid.
Diffstat (limited to 'gdb/common')
-rw-r--r-- | gdb/common/ptid.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/gdb/common/ptid.c b/gdb/common/ptid.c index 05c0db1..b56971b 100644 --- a/gdb/common/ptid.c +++ b/gdb/common/ptid.c @@ -97,10 +97,6 @@ ptid_is_pid (ptid_t ptid) int ptid_lwp_p (ptid_t ptid) { - if (ptid_equal (minus_one_ptid, ptid) - || ptid_equal (null_ptid, ptid)) - return 0; - return (ptid_get_lwp (ptid) != 0); } @@ -109,10 +105,6 @@ ptid_lwp_p (ptid_t ptid) int ptid_tid_p (ptid_t ptid) { - if (ptid_equal (minus_one_ptid, ptid) - || ptid_equal (null_ptid, ptid)) - return 0; - return (ptid_get_tid (ptid) != 0); } |