Remove reset_ecs and execution_control_state::reset
authorTom Tromey <tom@tromey.com>
Sun, 20 Nov 2022 22:08:06 +0000 (15:08 -0700)
committerTom Tromey <tom@tromey.com>
Mon, 28 Nov 2022 19:19:42 +0000 (12:19 -0700)
commitaa563d160d5e31861850e7e685df8c494fc9e307
tree00d1affddfd109bc3f66013538aabd2c1d3959ae
parent79d403654266c747703359e4b4e7f3931fb53f99
Remove reset_ecs and execution_control_state::reset

I noticed that execution_control_state has a 'reset' method, and
there's also a 'reset_ecs' function that calls it.  This patch cleans
this area up a little by adding a parameter to the constructor and (a
change Simon suggested) removing the reset method.  Some extraneous
variables are also removed, like:

-      struct execution_control_state ecss;
-      struct execution_control_state *ecs = &ecss;

Here 'ecs' is never changed, so this patch removes it entirely in
favor of just using the object everywhere.

Regression tested on x86-64 Fedora 34.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
gdb/infrun.c