Cleanup #1

Merged
merged 9 commits into from Apr 30, 2012

2 participants

@conorsch

Did a bit of small-form refactoring, e.g. added comments, renamed variables to be a bit more intuitive. Overall I'm truly surprised at how succinct you were able to get this (thank the gods for CPAN!). Thanks and I'll be using this a lot myself I think. =)

@Matael
Owner

Ok, I admit this wasn't a good idea ;) Thanks !

=) Once I stopped using foo/bar all over the place, eliding "data" and similarly ambiguous names was the next step for me. To legibility—and beyond!

Owner

If I have time, i'll try to write a small Vim function/script to check for well known ambiguous names before saving. This could really help me.

@Matael Matael merged commit 6abf7c7 into Matael:master Apr 30, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment