diff options
| author | Tom Tromey <tromey@adacore.com> | 2023-04-27 13:25:27 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-04-28 11:05:45 -0600 |
| commit | a38b832238eecf6611a338b39bb8f03f2968c26d (patch) | |
| tree | 385a0c6857a295638ca7d81a1440bb5b0e841c53 /gdb/python | |
| parent | ebb83b77a74a4f31d7fba7e892599c4552aad137 (diff) | |
| download | binutils-a38b832238eecf6611a338b39bb8f03f2968c26d.zip binutils-a38b832238eecf6611a338b39bb8f03f2968c26d.tar.gz binutils-a38b832238eecf6611a338b39bb8f03f2968c26d.tar.bz2 | |
Do not change type in get_discrete_low_bound
get_discrete_low_bound has this code:
/* Set unsigned indicator if warranted. */
if (low >= 0)
type->set_is_unsigned (true);
It's bad to modify a type in a getter like this, so this patch removes
this code. FWIW I looked and this code has been there since at least
1999 (it was in the initial sourceware import).
Types in general would benefit from const-ification, which would
probably reveal more code like this, but I haven't attempted that.
Regression tested on x86-64 Fedora 36.
Reviewed-by: Kevin Buettner <kevinb@redhat.com>
Diffstat (limited to 'gdb/python')
0 files changed, 0 insertions, 0 deletions