peterfeicht / impy Goto Github PK

When accidentally pressing, for example, an arrow key while entering a command on the VCP, the sent escape sequence is just echoed back and added to the command line, resulting in the command not being recognized.
This could be improved by just ignoring escape sequences altogether.

Better yet, certain escape sequences could be handled (for example, the up arrow could restore the previously executed command). This would probably also call for line editing capabilities, which
would be a lot of work I suppose.

Strict aliasing doesn't allow something like this: uint32_t real = *((uint32_t *)&tmp.Real);

This is used pervasively in Convert_PolarAscii (for hex number format) and Convert_PolarBinary (for endian reversal) and will probably break when optimization is turned on in a release build.

Either the Eclipse project needs to be changed to disable strict aliasing or the code needs to be changed (maybe using unions for all the structure fields to access them as uint32_t).