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

[1.1.x] Allow asterisks in G-Code commands #8196

Merged

Conversation

thinkyhead
Copy link
Member

Addressing #6655

Currently asterisks in a G-code command will confuse the parser. This change allows asterisks in a G-code command, and removes the error check assuming an asterisk without a line number is bad.

@Roxy-3D
Copy link
Member

Roxy-3D commented Nov 1, 2017

Let's also make the G-code parser smart enough to be tolerant of G's, M's, and N's in the command line.

@thinkyhead thinkyhead merged commit 1b68ee8 into MarlinFirmware:bugfix-1.1.x Nov 1, 2017
@thinkyhead thinkyhead deleted the bf1_asterisk_not_special branch November 1, 2017 02:53
@Roxy-3D
Copy link
Member

Roxy-3D commented Nov 1, 2017

Well OK... I'll start using Gxx *G {...} *N{...}

Surely.... That is acceptable.

@thinkyhead
Copy link
Member Author

This PR primarily relates to M117, M118, and other codes that take strings. It was never about parameters.

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.

None yet

2 participants