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

Patch 3 #4

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Patch 3 #4

wants to merge 10 commits into from

Conversation

s7560m
Copy link
Contributor

@s7560m s7560m commented Jun 30, 2015

Merging patch 3. This will clean and polish the code up efficiently, saving 8 lines of code from it. The multiple amount of commits is just me attempting to fix the strange indentation that github uses. It also fixes a parsing error in the previous script.

s7560m added 10 commits June 30, 2015 17:35
Let's use ternary operators for this instead of typical conditional value checkers. This saves quite a bit of space. This update also fixes a parsing mishap back at the key_stroke_e block.
Weird indentation issues.
Finally fixed indentation issues.
Changed multiple things to increase efficience of code. Diminished multiple lines of code. This is the final commit that basically polishes up the code from those multiple nested if statement blocks and uses multiple ternary operators to make it look more polished. Looks good so far.
Forgot to set bool to true.
Changed the conditions to ternary operators. Saved multiple lines, and there should be no debugging errors. Not exactly a recursive loop, but something that polishes the code and saves space.
I apologize for my clumsiness; wow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant