wso2-dev/carbon-business-messaging — pull request #80 by hastef88 (2014-12-15 07:18:51+00:00)
Title: Adding further review comments on my contributions upto now

Adding method comments, logging errors before throwing, using meaningful variable names.

Convert AndesConfigurationManager into a static class instead of a singleton one.

Add constants to define common errors during parsing configurations.

Use uniform method naming conventions in AndesConfigurationManager.

Handle the rare scenario of having an invalid default value. This exception is now handled inline instead of throwing outside. A null value is returned if this happens. (Given that its our responsibility to define semantically and type-wise correct default values, this scenario should ideally never happen.)