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

Add simplifyIfStatements Module and Tests #92

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Add simplifyIfStatements Module and Tests #92

merged 2 commits into from
Sep 11, 2023

Conversation

BenBaryoPX
Copy link
Collaborator

@BenBaryoPX BenBaryoPX commented Sep 11, 2023 •
edited
Loading

Solves an issue mentioned in #90

  • Remove redundant clauses from the if statement
  • Reverse logic when the consequent clause is empty and the alternate clause is not
  • Replace statement with test expression if all clauses are empty

@BenBaryoPX BenBaryoPX self-assigned this Sep 11, 2023
@BenBaryoPX BenBaryoPX merged commit 4c47043 into main Sep 11, 2023
2 checks passed
@BenBaryoPX BenBaryoPX deleted the add_simplify_if_statements_module branch September 11, 2023 13:58
@BenBaryoPX BenBaryoPX mentioned this pull request Sep 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@BenBaryoPX BenBaryoPX

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@BenBaryoPX