RFR: 8013839: Enhance Logger API for handling of resource bundles
Daniel Fuchs
daniel.fuchs at oracle.com
Fri Oct 4 14:44:42 UTC 2013
Hi Mandy,
On 10/3/13 9:29 PM, Mandy Chung wrote:
>
> test/java/util/ResourceBundle/getBaseBundleName/TestGetBaseBundleName.java
>
> 109 return new Vector<String>(java.util.Arrays.asList(
> 110 new String[] {"dummy"})).elements();
>
> Could you use Collections.enumeration?
Excellent! Yes I will.
> test/java/util/logging/Logger/logrb/TestLogrbResourceBundle.java
>
> 194 if (foobaz.getResourceBundleName() != null) {
> 195 throw new RuntimeException("Unexpected bundle: "
> 196 + foobar.getResourceBundle());
> 197 }
>
> I guess you meant to print foobar.getResourceBundleName() in line 196.
Oops - yes - thanks for catching that.
> The test cases are good. It'd be good to add some comments to
> describe what each case verifies when appropriate.
>
> You can make these changes before you push. No need for a new webrev.
OK - thanks
-- daniel
>
> thanks
> Mandy
>
More information about the core-libs-dev
mailing list