enriquein/JavaSetterGetter — pull request #7 by chrispwood (2013-03-16 15:44:58+00:00)
Title: Removed whitespace and added const getters

Enrique, thanks for creating this package! I am really enjoying it.

I made a couple changes that I thought I would share back for your review. The first provides accessors for final objects. Although strange, there is a rationale to accessors for constants in java: namely, javac inlines consts from 3rd party libraries unless they are encapsulated by accessors. A more thorough discussion can be found here: http://stackoverflow.com/a/10047926.

Second, when you don't select a valid instance member, the code prints out a couple lines of whitespace. I just added in a check to bail before the print to make sure no unneeded whitspace was printed.

Again, love the package - and it's another piece of ammunition to convince my colleagues to drop eclipse!

Regards,
Chris