diff --git a/Marlin/G26_Mesh_Validation_Tool.cpp b/Marlin/G26_Mesh_Validation_Tool.cpp
index 0cf41473598c0da77766046416d0c432c6e3ca06..e4fa01dd3c950cee51fa78f61753f296b8592778 100644
--- a/Marlin/G26_Mesh_Validation_Tool.cpp
+++ b/Marlin/G26_Mesh_Validation_Tool.cpp
@@ -686,7 +686,7 @@
}
}
- if (code_seen('N') || code_seen('n')) {
+ if (code_seen('N') || code_seen('n')) { // Warning! Use of 'N' / lowercase flouts established standards.
nozzle = code_value_float();
if (!WITHIN(nozzle, 0.1, 1.0)) {
SERIAL_PROTOCOLLNPGM("?Specified nozzle size not plausible.");
@@ -728,7 +728,7 @@
}
}
- if (code_seen('M')) {
+ if (code_seen('M')) { // Warning! Use of 'M' flouts established standards.
randomSeed(millis());
// This setting will persist for the next G26
random_deviation = code_has_value() ? code_value_float() : 50.0;
diff --git a/Marlin/ubl_G29.cpp b/Marlin/ubl_G29.cpp
index a44e9ee123f337b6585f10f09a59856c9b2c77e9..63f90461957c9b72802e2de21d22e7bff004c119 100644
--- a/Marlin/ubl_G29.cpp
+++ b/Marlin/ubl_G29.cpp
@@ -328,7 +328,8 @@
return;
}
- if (!code_seen('N') && axis_unhomed_error(true, true, true)) // Don't allow auto-leveling without homing first
+ // Don't allow auto-leveling without homing first
+ if (!code_seen('N') && axis_unhomed_error(true, true, true)) // Warning! Use of 'N' flouts established standards.
home_all_axes();
if (g29_parameter_parsing()) return; // abort if parsing the simple parameters causes a problem,
@@ -385,7 +386,7 @@
if (code_seen('J')) {
ubl.save_ubl_active_state_and_disable();
- ubl.tilt_mesh_based_on_probed_grid(code_seen('O') || code_seen('M'));
+ ubl.tilt_mesh_based_on_probed_grid(code_seen('O') || code_seen('M')); // Warning! Use of 'M' flouts established standards.
ubl.restore_ubl_active_state_and_leave();
}
@@ -419,7 +420,7 @@
SERIAL_PROTOCOLLNPGM(").\n");
}
ubl.probe_entire_mesh(x_pos + X_PROBE_OFFSET_FROM_EXTRUDER, y_pos + Y_PROBE_OFFSET_FROM_EXTRUDER,
- code_seen('O') || code_seen('M'), code_seen('E'), code_seen('U'));
+ code_seen('O') || code_seen('M'), code_seen('E'), code_seen('U')); // Warning! Use of 'M' flouts established standards.
break;
case 2: {
@@ -468,7 +469,7 @@
return;
}
- manually_probe_remaining_mesh(x_pos, y_pos, height, card_thickness, code_seen('O') || code_seen('M'));
+ manually_probe_remaining_mesh(x_pos, y_pos, height, card_thickness, code_seen('O') || code_seen('M')); // Warning! Use of 'M' flouts established standards.
SERIAL_PROTOCOLLNPGM("G29 P2 finished.");
} break;
@@ -504,7 +505,7 @@
//
// Fine Tune (i.e., Edit) the Mesh
//
- fine_tune_mesh(x_pos, y_pos, code_seen('O') || code_seen('M'));
+ fine_tune_mesh(x_pos, y_pos, code_seen('O') || code_seen('M')); // Warning! Use of 'M' flouts established standards.
break;
case 5: ubl.find_mean_mesh_height(); break;
@@ -549,7 +550,7 @@
// When we are fully debugged, the EEPROM dump command will get deleted also. But
// right now, it is good to have the extra information. Soon... we prune this.
//
- if (code_seen('j')) g29_eeprom_dump(); // EEPROM Dump
+ if (code_seen('j')) g29_eeprom_dump(); // Warning! Use of lowercase flouts established standards.
//
// When we are fully debugged, this may go away. But there are some valid
@@ -613,7 +614,7 @@
SERIAL_PROTOCOLLNPGM("Done.\n");
}
- if (code_seen('O') || code_seen('M'))
+ if (code_seen('O') || code_seen('M')) // Warning! Use of 'M' flouts established standards.
ubl.display_map(code_has_value() ? code_value_int() : 0);
if (code_seen('Z')) {
@@ -1128,8 +1129,8 @@
SERIAL_PROTOCOLLNPGM("Invalid map type.\n");
return UBL_ERR;
}
-
- if (code_seen('M')) { // Check if a map type was specified
+ // Check if a map type was specified
+ if (code_seen('M')) { // Warning! Use of 'M' flouts established standards.
map_type = code_has_value() ? code_value_int() : 0;
if (!WITHIN(map_type, 0, 1)) {
SERIAL_PROTOCOLLNPGM("Invalid map type.\n");