[m-rev.] for review: fix octal encoding bug in extras/graphics/mercury_allegro/samples

Julien Fischer jfischer at opturion.com
Wed Jan 5 19:52:36 AEDT 2022


Hi Fabrice,

Thanks for that.

On Wed, 5 Jan 2022, Fabrice Nicol wrote:

> A final modest proposal for review by anyone.
>
> -------------------------------------------------------------
>
> Bug issue (fixed):
>
> samples/demo/title.m:
>
> samples/mandel/mandel.m:
>
>         Compilation crashes due to illegal (perhaps obsolete) octal encoding
>
>         of 'Escape' Ascii character.
>
> ---------------------------------------------------------
>
> Proposed fix:
>
>         - Change illegal '\033' into legal 0o033

The fix is to terminate the octal escape correctly, e.g. '\033\'
I'v committed a modified verison of your change that does that.

Julien.


More information about the reviews mailing list