aboutsummaryrefslogtreecommitdiff
path: root/gdb/eval.c
diff options
context:
space:
mode:
authorFred Fish <fnf@specifix.com>2001-12-10 23:05:00 +0000
committerFred Fish <fnf@specifix.com>2001-12-10 23:05:00 +0000
commit0478d61cc28c9072f5261920744b734d17202862 (patch)
tree340c7f5f7aa4679cbbdaea49b8bd3fcded0926ba /gdb/eval.c
parent709ec538af69bad6c603acc9b985aa900fea0ff2 (diff)
downloadgdb-0478d61cc28c9072f5261920744b734d17202862.zip
gdb-0478d61cc28c9072f5261920744b734d17202862.tar.gz
gdb-0478d61cc28c9072f5261920744b734d17202862.tar.bz2
Approved by Jim Blandy:
2001-12-10 Fred Fish <fnf@redhat.com> * values.c (value_fn_field): Add physname variable. Use a minimal symbol if we don't find a full symbol. Remove setting of the new value's type since that was already done by allocate_value(). Remove obsolete commented out error call since callees need to handle a NULL return, which is possible result not an error. * eval.c (evaluate_subexp_standard): Move check for inlined functions to precede attempt to dereference a NULL argvec[0].
Diffstat (limited to 'gdb/eval.c')
-rw-r--r--gdb/eval.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/eval.c b/gdb/eval.c
index c15b236..8a41d30 100644
--- a/gdb/eval.c
+++ b/gdb/eval.c
@@ -917,6 +917,8 @@ evaluate_subexp_standard (struct type *expect_type,
if (noside == EVAL_SKIP)
goto nosideret;
+ if (argvec[0] == NULL)
+ error ("Cannot evaluate function -- may be inlined");
if (noside == EVAL_AVOID_SIDE_EFFECTS)
{
/* If the return type doesn't look like a function type, call an
@@ -934,8 +936,6 @@ evaluate_subexp_standard (struct type *expect_type,
else
error ("Expression of type other than \"Function returning ...\" used as function");
}
- if (argvec[0] == NULL)
- error ("Cannot evaluate function -- may be inlined");
return call_function_by_hand (argvec[0], nargs, argvec + 1);
/* pai: FIXME save value from call_function_by_hand, then adjust pc by adjust_fn_pc if +ve */