Hello,

Most of this is my code, so I should comment on this.

On Aug 30, Sreenivas Subramoney wrote:
> Below are the bugs I have encountered and proposed fixes.
> 
> 1)
> 
> The loop in ../java/util/Properties::getProperty() does not correctly search
> the default of the defalut property list. "prop" must be correctly
> initialized and updated, and also the method "get()" should be called on the
> "prop" object.

Yes, you are right. I fixed it.

> 2)
> 
> The functions ResourceBundle::getBundle(),  ResourceBundle::getObject() and
> ResourceBundle.getString() throw MissingResourceException and calls to these
> should be surrounded by a "try...catch" block if the caller function itself
> does not throw it. I have added them where they were absent.

I don't think so.  MissingResourceException is only thrown, if the
resources can't be found, which means, that classpath is not installed
correctly, or there is a bug in classpath's resources.  I think it is
good, if the exception is passed through.  

  Jochen

Reply via email to