Why are these m_ prefixed and other vars aren't? I think our convention in core code is to make all member vars prefixed, but our tools code isn't that consistent. Still, it would be nice to go one way or another.

Also some comments explaining what these vars do would be awesome. Some are more obvious than others.