[PATCH 2/6] Convert fatal to gdb_assert in both i386_dr_low_set_addr
Pedro Alves
palves@redhat.com
Wed Aug 20 16:09:00 GMT 2014
On 08/06/2014 04:58 PM, Gary Benson wrote:
> This commit converts if..fatal checks in both i386_dr_low_set_addr
> implementations to gdb_asserts. It's not obvious from the context,
> but the conditional in both cases is changed to match the equivalent
> conditional in the i386_dr_low_get_addr implementations. Nothing
> fundamental has changed because DR_FIRSTADDR is zero. This commit
> also removes a vague comment in Linux i386_dr_low_get_addr. I could
> have reworded the comment (and replicated it three times for the other
> identical assertions) but I think the existence of specific functions
> for the status and control registers makes it fairly obvious what is
> going on.
>
OK, with ...
> gdb/gdbserver/
> 2014-08-06 Gary Benson <gbenson@redhat.com>
>
> * linux-x86-low.c (i386_dr_low_set_addr): Replace check with
> gdb_assert.
> (i386_dr_low_get_addr): Remove vague comment.
> (win32-i386-low.c): Replace check with gdb_assert.
... the win32 entry fixed.
Thanks,
Pedro Alves
More information about the Gdb-patches
mailing list