Skip to content

[1.1.x] Fix Dual X Carriage tool change crash#11425

Merged
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-1.1.xfrom
silentninja1:1.1.x-IDEX-Crash-Fix-Rev2
Aug 4, 2018
Merged

[1.1.x] Fix Dual X Carriage tool change crash#11425
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-1.1.xfrom
silentninja1:1.1.x-IDEX-Crash-Fix-Rev2

Conversation

@silentninja1
Copy link
Contributor

Followup to Issue #11252 with a different method that should be adaptable enough for other manufacturers. Tested on Formbot Trex2+.

@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch from 6eca160 to 5e5dec3 Compare August 3, 2018 22:30
@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch from 5e5dec3 to 3162937 Compare August 3, 2018 22:42
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should use tmp_extruder == 1 here instead, since the result is that the X2 items are selected for use.

@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch from 3162937 to 8cf0221 Compare August 3, 2018 22:49
@thinkyhead thinkyhead changed the title 1.1.x idex crash fix rev2 [1.1.x] Fix Dual X Carriage tool change crash Aug 3, 2018
@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch from 8cf0221 to c15aee0 Compare August 3, 2018 23:17
@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch 3 times, most recently from 96c3315 to 2749b0b Compare August 4, 2018 00:06
@thinkyhead thinkyhead force-pushed the 1.1.x-IDEX-Crash-Fix-Rev2 branch from 2749b0b to 77fd7de Compare August 4, 2018 01:01
@thinkyhead thinkyhead merged commit 4c4fffc into MarlinFirmware:bugfix-1.1.x Aug 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants