From: Jim Morris <morris@wolfman.com>
Date: Sun, 13 Dec 2015 23:40:34 +0000 (-0800)
Subject: fix G53
X-Git-Url: https://git.hcoop.net/clinton/Smoothieware.git/commitdiff_plain/5d2319a9ab4178286b7a35a9f9467482926b9ad2

fix G53
---

diff --git a/src/modules/communication/GcodeDispatch.cpp b/src/modules/communication/GcodeDispatch.cpp
index 2e75e4e7..37d6d90e 100644
--- a/src/modules/communication/GcodeDispatch.cpp
+++ b/src/modules/communication/GcodeDispatch.cpp
@@ -150,15 +150,23 @@ try_again:
                     if(gcode->has_g) {
                         if(gcode->g == 53) { // G53 makes next movement command use machine coordinates
                             // this is ugly to implement as there may or may not be a G0/G1 on the same line
+                            // valid vesion seem to include G53 G0 X1 Y2 Z3 G53 X1 Y2
                             if(possible_command.empty()) {
-                                // use last gcode G1 or G0 if none on the line
+                                // use last gcode G1 or G0 if none on the line, and pass through as if it was a G0/G1
                                 // TODO it is really an error if the last is not G0 or G1
+                                if(last_g != 0 && last_g != 1) {
+                                    delete gcode;
+                                    new_message.stream->printf("ok - Invalid G53\r\n");
+                                    return;
+                                }
+                                // use last G0 or G1
                                 gcode->g= last_g;
 
                             }else{
                                 delete gcode;
                                 // extract next G0/G1 from the rest of the line, ignore if it is not one of these
                                 gcode = new Gcode(possible_command, new_message.stream);
+                                possible_command= "";
                                 if(!gcode->has_g || gcode->g > 1) {
                                     // not G0 or G1 so ignore it as it is invalid
                                     delete gcode;
@@ -168,7 +176,10 @@ try_again:
                             }
                             // makes it handle the parameters as a machine position
                             THEKERNEL->robot->next_command_is_MCS= true;
+
                         }
+
+                        // remember last modal g code
                         last_g= gcode->g;
                     }
 
diff --git a/src/modules/robot/Robot.cpp b/src/modules/robot/Robot.cpp
index a0d9fc9e..ed943a6c 100644
--- a/src/modules/robot/Robot.cpp
+++ b/src/modules/robot/Robot.cpp
@@ -613,6 +613,7 @@ void Robot::on_gcode_received(void *argument)
     next_command_is_MCS = false; // must be on same line as G0 or G1
 }
 
+// process a G0/G1/G2/G3
 void Robot::process_move(Gcode *gcode)
 {
         // we have a G0/G1/G2/G3 so extract parameters and apply offsets to get machine coordinate target