Browse Source

Remove hidden dependencies on destination. Fixes #10686. (#10691)

pull/1/head
ManuelMcLure 6 years ago
committed by Scott Lahteine
parent
commit
613a3be167
  1. 2
      Marlin/src/gcode/motion/G2_G3.cpp
  2. 8
      Marlin/src/gcode/motion/G5.cpp

2
Marlin/src/gcode/motion/G2_G3.cpp

@ -226,7 +226,7 @@ void plan_arc(
// As far as the parser is concerned, the position is now == target. In reality the // As far as the parser is concerned, the position is now == target. In reality the
// motion control system might still be processing the action and the real tool position // motion control system might still be processing the action and the real tool position
// in any intermediate location. // in any intermediate location.
set_current_from_destination(); COPY(current_position, cart);
} // plan_arc } // plan_arc
/** /**

8
Marlin/src/gcode/motion/G5.cpp

@ -27,13 +27,13 @@
#include "../../module/motion.h" #include "../../module/motion.h"
#include "../../module/planner_bezier.h" #include "../../module/planner_bezier.h"
void plan_cubic_move(const float (&offset)[4]) { void plan_cubic_move(const float (&cart)[XYZE], const float (&offset)[4]) {
cubic_b_spline(current_position, destination, offset, MMS_SCALED(feedrate_mm_s), active_extruder); cubic_b_spline(current_position, cart, offset, MMS_SCALED(feedrate_mm_s), active_extruder);
// As far as the parser is concerned, the position is now == destination. In reality the // As far as the parser is concerned, the position is now == destination. In reality the
// motion control system might still be processing the action and the real tool position // motion control system might still be processing the action and the real tool position
// in any intermediate location. // in any intermediate location.
set_current_from_destination(); COPY(current_position, cart);
} }
/** /**
@ -69,7 +69,7 @@ void GcodeSuite::G5() {
parser.linearval('Q') parser.linearval('Q')
}; };
plan_cubic_move(offset); plan_cubic_move(destination, offset);
} }
} }

Loading…
Cancel
Save