8050 loader: menu.rc small typos in menu 8051 loader: switching BE should unset the previously set env
Review Request #433 - Created April 10, 2017 and submitted
8050 loader: menu.rc small typos in menu
8051 loader: switching BE should unset the previously set env
In the last two files there are extra whitespace showing up here as a red box behind nearly all empty comment lines /
Normally this would not matter. However, last year I noticed there were some high order ASCII in the comments of the beadm.4th file. I don't see any red boxes here for beadm.4th now though. Probably broken UTF-8 related. I suspect cut/paste a polluted comment line. I was concerned the Forth interpreter could be made to choke on it in the future to introduce an attack surface. They might still be lurking.
Otherwise the coding looks good.
ok more boot/forth/beadm.4th
Three \ comment lines have two high-order ASCII characters immediately
following the \ character which might cause a problem with the Forth
interpreter. The two characters are a box drawing single line downward
intersection like a 'T' (IBM VGA font16 0xc2), and an acute accent lowercase
'a' (maybe 0xe1).
cleaning up the whitespace issues.
Revision 2 (+59 -44)