RE: [PATCH v2 07/11] gdbserver: introduce and use regcache::set_register_status

Aktemur, Tankut Baris
2025-01-09T12:50:46Z

On Tuesday, January 7, 2025 6:02 AM, Simon Marchi wrote:
> On 2024-12-30 05:49, Tankut Baris Aktemur wrote:
> > Introduce and use a setter method in regcache to set the status of a
> > register. There already exists get_register_status. So, it made
> > sense to add the setter to control access to the register_status
> > field.
> >
> > In two places, we also do cosmetic improvements to for-loops.
>
> With this, it looks like we're one call away from being able to make
> `register_status` private.
>
> Approved-By: Simon Marchi <simon.marchi@efficios.com>
>
> Simon

Thanks. I pushed this as is, relying on the other in-review patches
to do the remaining refactorings.

-Baris

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Sean Fennelly, Jeffrey Schneiderman, Tiffany Doon Silva