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

Aktemur, Tankut Baris tankut.baris.aktemur@intel.com
Thu Jan 9 12:49:15 GMT 2025


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
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928


More information about the Gdb-patches mailing list