[m-rev.] for post-commit review: add color in det_report.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Tue Apr 30 19:35:40 AEST 2024


On 2024-04-29 20:57 +10:00 AEST, "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
>> "int" and "uint8" are now colored, but nothing else is.  I think there
>> is a case to be made that the variable "N", which is the subject of the
>> message should stand out mode.
> 
> That is a good idea. Internally, we could call that color "color_subject",
> as it denotes the subject of the error messsage. Does anyone have
> a better name in mind? And what should the default color shade for be?
> Some kind of blue, given that red, green and yellow are already in use
> for other things? Or should we rejig the existing colors?

Unless I hear some other opinions by then, about this time tomorrow I intend to

- add color_subject, with the default shade being a light cyan, probably 8-bit color 87, and

- rename all the .exp files in tests/warnings to .err_exp files.

Zoltan.


More information about the reviews mailing list