On 9/3/2014 6:57 PM, Fraser Tweedale wrote:
I don't know the history either, but the SimpleProperties allows you to specify the default values of some properties so what you get is the effective values. This might be useful for the server because the server works using the effective property values, but I'm not sure if the current code is actually using it. On the client side we should be dealing with the configured properties only.
There are probably some other differences such as adding a header on the top of the file and the property value encoding. I don't think we need it on the client side though. Also note that some of these codes are quite old, and newer/better utility classes might have been added into the standard/common Java libraries since then, so in some cases it might be better to replace it if possible. Usually the hard part is testing it to make sure we're not breaking the existing functionality.
`Properties' has both default values and header comments, FWIW. Anyhow I'll leave the refactor for now and file a ticket, and proceed with Properties for the client-side.
You're right, I didn't see that. Yeah, we'll leave that for now until we have time to look at it in more details. -- Endi S. Dewata