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.

Sign up for GitHub

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

Jump to bottom

use TERN0 or TERN1 instead of TERN(option, A, 0|1) #27036

Merged
merged 3 commits into from
May 15, 2024
Merged

use TERN0 or TERN1 instead of TERN(option, A, 0|1) #27036

merged 3 commits into from
May 15, 2024

Conversation

ellensp
Copy link
Contributor

@ellensp ellensp commented Apr 30, 2024

Description

Code tidy up

Requirements

None

Benefits

More concise code

@thinkyhead
Copy link
Member

It seems a good idea to keep the TERN(a, b) where b refers to some "meaningful" value, then just replace the ones where the 1 / 0 is "optional" or refers to an "all or none" kind of situation. Semantics, amirite?

@thinkyhead thinkyhead merged commit 3dc437a into MarlinFirmware:bugfix-2.1.x May 15, 2024
62 checks passed
@ellensp ellensp deleted the add-TERN0-and-TERN1 branch May 16, 2024 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR: General Cleanup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ellensp @thinkyhead