From 49205f92ff7a2ee8d3ed5dc9992afb74d82c597f Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Tue, 26 Nov 2024 17:32:40 -0500 Subject: gcode: Don't silently discard characters inside a command name Don't silently drop leading numbers and unusual characters at the start of a command - for example, don't interpret '99M88' as 'M88'. Don't silently drop spaces in a command - for example, don't interpret "M 101" as the command "M101". Doing so will cause other parts of the code (such as get_raw_command_parameters() ) to not work properly. Signed-off-by: Kevin O'Connor --- docs/Config_Changes.md | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'docs') diff --git a/docs/Config_Changes.md b/docs/Config_Changes.md index 5535dc85..ecac5a7a 100644 --- a/docs/Config_Changes.md +++ b/docs/Config_Changes.md @@ -8,6 +8,12 @@ All dates in this document are approximate. ## Changes +20241201: In some cases Klipper may have ignored leading characters or +spaces in a traditional G-Code command. For example, "99M123" may have +been interpreted as "M123" and "M 321" may have been interpreted as +"M321". Klipper will now report these cases with an "Unknown command" +warning. + 20241112: Option `CHIPS=` in `TEST_RESONANCES` and `SHAPER_CALIBRATE` requires specifying the full name(s) of the accel chip(s). For example, `adxl345 rpi` instead of short name - `rpi`. -- cgit v1.2.3-70-g09d2