Byzantium/Byzantium — pull request #148 by shanel (2012-07-17 03:59:48+00:00)
Title: Requisite changes for Issue 129 (and a whole lot of refactoring)

I kind of ended up going a bit down the rabbit hole here. All the global variables and whatnot were making it pretty rough to test my changes so I had to get rid of those. I also moved to using argparse instead of getopt since it is a lot cleaner (though to be honest I would really prefer gflags). I added a single test but plan on adding more. There were some epically big methods in some of the files so I broke them up a bit.

I ran the code through clonedigger and cleaned up a fair bit of repetitive code as well. I also did a bit of linting here and there based on pylint and the Google Python style guide (http://google-styleguide.googlecode.com/svn/trunk/pyguide.html) though there is a fair bit more work to do along the linting lines. ;)

If this is way too much stuff going on in a single pull request I can see what I can do to break it out, but I tend to find refactoring and testing to be the best way to get to know a code base so I figured I would try it out.

Cheers!

-Shane