Re: classpath ./ChangeLog ./THANKYOU java/util/Reso...

[Brian Jones]

Tom Tromey <address@hidden> writes:

> >>>>> "Brian" == Brian Jones <address@hidden> writes:
> 
> Brian>        2003-01-05  Julian Dolby  <address@hidden>
> Brian>        * java/util/ResourceBundle.java (tryBundle): If an exception
> Brian>        occurs, do not try to use the bundle.
> 
> This part of the patch looks like this:
> 
> @@ -509,6 +523,7 @@
>      catch (Exception ex)
>        {
>          // ignore them all
> +     foundBundle = null;
>        }
>      if (foundBundle == null)
>        {
> 
> I think this is unnecessary.  The assignment to `foundBundle' will
> never happen if there is an exception.  I'm inclined to revert this
> part, as it is a libgcj/classpath divergence and is a little easier to
> fix here.

This is true, but the code is harmless and I was inclined to believe
necessary for JRVM for some reason.  If it isn't needed (Dolby?) then
surely revert that part.

Thanks,
Brian
-- 
Brian Jones <address@hidden>