[m-rev.] for review: more info in diagnostics for subtype errors

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Jun 30 19:18:59 AEST 2021


2021-06-30 14:55 GMT+10:00 "Peter Wang" <novalazy at gmail.com>:
>> +subtype_abstract.m:012: Error: the type definition for `citrus'/0,
>> +subtype_abstract.m:012:   the super type of `lemon'/0,
>> +subtype_abstract.m:012:   is not visible here.
> 
> Perhaps it would be an improvement to add the word "declared", as in:
> 
>     Error: the type definition for `citrus'/0,
>       the declared super type of `lemon'/0,
>       is not visible here.

In the presence of a longer chain of supertypes, are you proposing
to add "declared" to just the first supertype, or all supertypes in the chain?

> It's not necessary, though.

OK, I am committing the diff as is with your two corrections above this.
I will add "declared" in a separate commit once you tell me your preference.

BTW, is anyone going to review the uint->uintN diff? My change to du_type_layout.m
needs it.

Zoltan.

Zoltan.



More information about the reviews mailing list