[m-rev.] for post-commit review: improve error messages for initialise/finalise
Peter Wang
novalazy at gmail.com
Mon May 24 15:14:09 AEST 2021
On Mon, 24 May 2021 03:39:01 +1000 "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
> For review by anyone.
>
> Zoltan.
> Fix error messages for initialise/finalise decls.
>
> compiler/make_hlds_passes.m:
> When rejecting a predicate due to a invalid signature, say what
> a valid signature looks like.
>
> Replace two cut-and-pasted predicates with one parameterised predicate.
>
> compiler/parse_mutable.m:
> Replace two different error messages for malformed name/arity in
> initialise and finalise declarations respectively with one message
> that follows the same patterns as many of our other error messages.
>
> tests/invalid/bad_finalise_decl.err_exp:
> tests/invalid/bad_initialise_decl.err_exp:
> Expect the updated error messages.
That looks fine.
Peter
More information about the reviews
mailing list