Clean up arguments to evaluate_subexp_do_call
authorTom Tromey <tom@tromey.com>
Wed, 16 Dec 2020 00:53:34 +0000 (17:53 -0700)
committerTom Tromey <tom@tromey.com>
Wed, 16 Dec 2020 00:53:34 +0000 (17:53 -0700)
commit1ab8280d7dbe01f51929d221621d38450f31895d
tree6ae791784db83a48b4cf9877e4f2a1ee481d3c92
parentcf608cc40c523537046efde52e9d68303e8bde00
Clean up arguments to evaluate_subexp_do_call

I noticed hat evaluate_subexp_do_call takes an array of arguments and
a count -- but, unlike the usual convention, the count does not
include the first element.

This patch changes this function to match call_function_by_hand --
passing the callee separately, and using an array_view for the
arguments.  This makes it simpler to understand.

Regression tested on x86-64 Fedora 28.

gdb/ChangeLog
2020-12-15  Tom Tromey  <tom@tromey.com>

* f-lang.c (evaluate_subexp_f): Update.
* expression.h (evaluate_subexp_do_call): Update.
* eval.c (evaluate_subexp_do_call): Add callee parameter.  Replace
nargs, argvec with array_view.
(evaluate_funcall): Update.
gdb/ChangeLog
gdb/eval.c
gdb/expression.h
gdb/f-lang.c