neurolabs/simpleconf — pull request #1 by magro (2011-12-23 16:48:23+00:00)
Title: Suggested changes after first review

Several stuff, details should be obviuos from commit messages.

Further comments:
* Logging is not that efficient (using String.format) but as loading properties is done once and not that performance critical I think it's ok.
* ApplicationPropertiesListener could provide more docs in the lines of usage instructions (see also my comment here https://github.com/neurolabs/simpleconf/commit/d1c40e8b84f8aad79e9e9e81d6f3bb4d733f979f#L3R23). But this is not that important if the README already provides usage instructions (and also because javadoc probably will not be hosted somewhere so it will not be the most recommended documentation). Therefore no change needed for ApplicationPropertiesListener.