Do not change type in get_discrete_low_bound
authorTom Tromey <tromey@adacore.com>
Thu, 27 Apr 2023 19:25:27 +0000 (13:25 -0600)
committerTom Tromey <tromey@adacore.com>
Fri, 28 Apr 2023 17:05:45 +0000 (11:05 -0600)
commita38b832238eecf6611a338b39bb8f03f2968c26d
tree385a0c6857a295638ca7d81a1440bb5b0e841c53
parentebb83b77a74a4f31d7fba7e892599c4552aad137
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>
gdb/gdbtypes.c