Skip to content

Cleanup#73

Merged
tleonhardt merged 2 commits intomasterfrom
cleanup
Mar 12, 2017
Merged

Cleanup#73
tleonhardt merged 2 commits intomasterfrom
cleanup

Conversation

@tleonhardt
Copy link
Copy Markdown
Member

Code cleanup

  • Moved some code that was located in the middle of stuff it wasn't related to
  • Added a bunch of comments, mostly to classes and methods to explain their purpose
  • Fixed a logic bug where postparsing_postcmd() was being called twice for unrecognized commands

- Reorganized some floating code that was located poorly in the middle of stuff it wasn't related to
- Added a bunch of comments
- Fixed a logic bug where postparsing_postcmd() was being called twice for unrecognized commands
Added a bunch of comments to try to explain why things exist and what they are doing.
@tleonhardt tleonhardt added this to the 0.7.1 milestone Mar 12, 2017
@tleonhardt tleonhardt self-assigned this Mar 12, 2017
@tleonhardt tleonhardt merged commit 3a8b3ba into master Mar 12, 2017
@tleonhardt tleonhardt deleted the cleanup branch March 12, 2017 22:58
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.

1 participant