[m-rev.] for review of the concept: shorten function symbol names in error_spec.m

Julien Fischer jfischer at opturion.com
Sat Apr 20 21:46:42 AEST 2024


On Sat, 20 Apr 2024, Zoltan Somogyi wrote:

> Shorten function symbols in error_specs.
> 
> compiler/error_spec.m:
>     Replace simplest_spec with spec, and simplest_no_context_spec with
>     no_ctxt_spec. There are now the most frequently used function symbols

s/There/These/

>     to create error specs, so their name should not make them out to be
>     the exception.
>
>     Replace simplest_msg with msg, and simplest_no_context_msg with
>     no_ctxt_msg for the same reason.
>
>     Abbreviate some of the long phase names.
> 
> compiler/*.m:
>     Conform to the changes above. Most of these changes were done by a script,
>     with minor manual tidying up, which consisted mostly of fitting code
>     constructing error specs onto fewer lines than before.

That's fine otherwise.

Julien.


More information about the reviews mailing list