Fix warnings: variables, not initialized or not set #337
CloCkWeRX
added
some commits
Dec 12, 2016
tatey
commented
Dec 13, 2016
|
Appreciate you fixing these tests. I'm just in the process of making a patch and saw all the warnings |
CloCkWeRX
added
some commits
Dec 13, 2016
|
Might need to have a think about the skip_writer or calculated changes; I added documentation but am not really sure if its enough. It does prevent double method definition though. |
nnc
commented
Dec 13, 2016
|
Thank you for this pull request @CloCkWeRX. It's so great to see all this work being done on Xeroizer! It may be easier and quicker for @waynerobinson to review and merge all these changes if you can split this into 3 pull request:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This isn't all of them, but it makes things look a lot less like: