aboutsummaryrefslogtreecommitdiff
path: root/gdb/value.c
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2021-03-08 07:27:57 -0700
committerTom Tromey <tom@tromey.com>2021-03-08 07:28:44 -0700
commit3dd93bf837b10e01864d7d223ae9d3f9b92df74e (patch)
tree6f48c2db566069ae33c55e883994927dcc1b570c /gdb/value.c
parent9c79936b3df018166d53869f7f2bb2909f129e51 (diff)
downloadgdb-3dd93bf837b10e01864d7d223ae9d3f9b92df74e.zip
gdb-3dd93bf837b10e01864d7d223ae9d3f9b92df74e.tar.gz
gdb-3dd93bf837b10e01864d7d223ae9d3f9b92df74e.tar.bz2
Remove some null checks
When not parsing for completion, parse_expression ensures that the resulting expression has operations. This patch removes a couple of unnecessary checks for this situation. gdb/ChangeLog 2021-03-08 Tom Tromey <tom@tromey.com> * printcmd.c (set_command): Remove null check. * value.c (init_if_undefined_command): Remove null check.
Diffstat (limited to 'gdb/value.c')
-rw-r--r--gdb/value.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/value.c b/gdb/value.c
index df0e5c1..9527186 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -2015,7 +2015,7 @@ init_if_undefined_command (const char* args, int from_tty)
/* Validate the expression.
Was the expression an assignment?
Or even an expression at all? */
- if (expr->op == nullptr || expr->first_opcode () != BINOP_ASSIGN)
+ if (expr->first_opcode () != BINOP_ASSIGN)
error (_("Init-if-undefined requires an assignment expression."));
/* Extract the variable from the parsed expression. */