[m-rev.] for post-commit review: extend "did you mean" suggestions to other kinds of entities
Julien Fischer
jfischer at opturion.com
Tue Sep 26 23:32:28 AEST 2023
On Tue, 26 Sep 2023, Zoltan Somogyi wrote:
> Extend "did you mean" suggestions beyond preds.
>
> compiler/typecheck_error_undef.m:
> Suggest possible "did you mean" names when reporting references
> to undefined functions, not just undefined predicates.
>
> Generalize the infrastructure used for predicate name suggestions
> to work for function name suggestions as well.
>
> Delete the code moved to error_spec.m.
>
> compiler/module_qual.qual_errors.m:
> When reporting an undefined entity name that has way to become defined
> without changing its name, suggest possible names.
>
> compiler/error_spec.m:
> Move the code for constructing "did you mean" suggestions here from
> typecheck_error_undef.m, to make it accessible from
> module_qual.qual_errors.m as well.
>
> compiler/hlds_cons.m:
> Add a way to retrieve the names of all the function symbols,
> for use by the new code in typecheck_error_undef.m.
>
> compiler/module_qual.id_set.m:
> Add a way to retrieve the names of all the ids in an id_set,
> for use by the new code in module_qual.qual_errors.m.
>
> Expand a old comment.
>
> tests/invalid/bug113.err_exp:
> tests/invalid/multiply_star.err_exp:
> tests/invalid/multiply_star.m:
> tests/invalid_nodepend/kind.err_exp:
> tests/invalid_nodepend/reserved_type_name.err_exp:
> Expect the "did you mean" messages we now generate.
That looks fine.
Julien.
More information about the reviews
mailing list