Kind of cutting it close here. Well I didn't accomplish a lot, but hopefully it helps a little. I ran out of time before I got anything more done. I'm submitting my changes in pieces so that you can pick and choose what you like and don't like. Many of these changes are superficial and subjective.
This first pull request is my "tidy" branch. It is intended just to clean up the code a little bit. Feel welcome to disagree with any of these changes. It's your module and your code and you ultimately have to be the one that likes it. :)
It may well be easier to merge this branch last since it may be less likely to conflict in that order. No guarantees though. You can experiment with what works. If you have any trouble merging please contact me. I would be more than happy to rebase. Thanks!
Kind of cutting it close here. Well I didn't accomplish a lot, but hopefully it helps a little. I ran out of time before I got anything more done. I'm submitting my changes in pieces so that you can pick and choose what you like and don't like. Many of these changes are superficial and subjective.
This first pull request is my "tidy" branch. It is intended just to clean up the code a little bit. Feel welcome to disagree with any of these changes. It's your module and your code and you ultimately have to be the one that likes it. :)
It may well be easier to merge this branch last since it may be less likely to conflict in that order. No guarantees though. You can experiment with what works. If you have any trouble merging please contact me. I would be more than happy to rebase. Thanks!