Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix empty Prepare menu #5647

Merged
merged 1 commit into from
Jan 12, 2017
Merged

Fix empty Prepare menu #5647

merged 1 commit into from
Jan 12, 2017

Conversation

manianac
Copy link
Contributor

@manianac manianac commented Jan 1, 2017

Fixes the blank prepare menu error. When using a single hotend, there is no for loop, so having a break statement would exit the function. While not great coding practice, its alright to loop all the hotends to check for heat.

The other option would be to add a

#if HOTENDS > 1
 break;
#endif

but I'll leave that up to @thinkyhead or whoever merges this.

I also renamed e to the index macro.

Use index macro instead of e
@manianac manianac changed the title Remove error causing break command Fix empty Prepare menu Jan 1, 2017
This was referenced Jan 3, 2017
@thinkyhead thinkyhead merged commit f8ae0fd into MarlinFirmware:RCBugFix Jan 12, 2017
@Blue-Marlin
Copy link
Contributor

@thinkyhead
Welcome back.

@ghost ghost mentioned this pull request Feb 14, 2017
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.

3 participants