]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
Fix regression in Ada do_full_match
authorTom Tromey <tromey@adacore.com>
Thu, 7 Jan 2021 14:02:46 +0000 (07:02 -0700)
committerTom Tromey <tromey@adacore.com>
Thu, 7 Jan 2021 14:06:26 +0000 (07:06 -0700)
commit959d6a673ea5f5d3f057165318cee8cea07fd5fc
tree2bd7911a42eb5b6cb530d76979894bed5be2f57b
parentdc20061339031e391f974ccccb71137b51730494
Fix regression in Ada do_full_match

An earlier patch to ada-lang.c:do_full_match introduced a subtle
change to the semantics.  The previous code did:

    -  if (strncmp (sym_name, search_name, search_name_len) == 0
    -      && is_name_suffix (sym_name + search_name_len))
    -    return true;
    -
    -  if (startswith (sym_name, "_ada_")

whereas the new code unconditionally skips a leading "_ada_".

The difference occurs if the lookup name itself starts with "_ada_".
In this case, the symbol won't match.

Normally this doesn't seem to be a problem.  However, it caused a
regression on one particular (internal) test case on one particular
platform.

This patch changes the code to handle this case.  I don't know how to
write a reliable test case for this, so no test is included.

2021-01-07  Tom Tromey  <tromey@adacore.com>

* ada-lang.c (do_full_match): Conditionally skip "_ada_" prefix.
gdb/ChangeLog
gdb/ada-lang.c