aboutsummaryrefslogtreecommitdiff
path: root/gdb/common
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@ericsson.com>2017-04-05 15:15:59 -0400
committerSimon Marchi <simon.marchi@ericsson.com>2017-04-05 15:15:59 -0400
commit4e9868d4e0c8e45505876901d22c021dd36972a8 (patch)
tree279804eb170424ac97824ec1540120a0763cbfda /gdb/common
parent9bf2a700667c53003ece783c05e8b355801105f2 (diff)
downloadgdb-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.c8
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);
}