RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel
Re: RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
Looking at the jdk7 source and comparing your change this looks OK. Would be good to get one more pair of eyes on this as well but I think you should be good to go Best Lance On Aug 23, 2013, at 10:41 AM, Daniel Fuchs wrote: Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
Re: RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
Hi Daniel, The fix looks good. It might be good to clarify the spec in the case when the given resourceBundleName parameter is null? Perhaps in the @params resourceBundleName of the getLogger(String, String) method. * @param resourceBundleName name of ResourceBundle to be used for localizing * messages for this logger. May be CODEnull/CODE if none of * the messages require localization. perhaps clarify that null only if the logger does not have a localization resource bundle set. thanks Mandy On 8/23/2013 7:41 AM, Daniel Fuchs wrote: Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel
Re: RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
On 8/23/13 5:50 PM, Mandy Chung wrote: Hi Daniel, The fix looks good. It might be good to clarify the spec in the case when the given resourceBundleName parameter is null? Perhaps in the @params resourceBundleName of the getLogger(String, String) method. * @param resourceBundleName name of ResourceBundle to be used for localizing * messages for this logger. May be CODEnull/CODE if none of * the messages require localization. perhaps clarify that null only if the logger does not have a localization resource bundle set. Thanks Lance and Mandy! I have updated the webrev as per your suggestion: http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.01/ -- daniel thanks Mandy On 8/23/2013 7:41 AM, Daniel Fuchs wrote: Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel
Re: RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
Thumbs up. thanks Mandy On 8/23/2013 10:26 AM, Daniel Fuchs wrote: On 8/23/13 5:50 PM, Mandy Chung wrote: Hi Daniel, The fix looks good. It might be good to clarify the spec in the case when the given resourceBundleName parameter is null? Perhaps in the @params resourceBundleName of the getLogger(String, String) method. * @param resourceBundleName name of ResourceBundle to be used for localizing * messages for this logger. May be CODEnull/CODE if none of * the messages require localization. perhaps clarify that null only if the logger does not have a localization resource bundle set. Thanks Lance and Mandy! I have updated the webrev as per your suggestion: http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.01/ -- daniel thanks Mandy On 8/23/2013 7:41 AM, Daniel Fuchs wrote: Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel
Re: RFR: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle
looks fine dan! On Aug 23, 2013, at 1:26 PM, Daniel Fuchs wrote: On 8/23/13 5:50 PM, Mandy Chung wrote: Hi Daniel, The fix looks good. It might be good to clarify the spec in the case when the given resourceBundleName parameter is null? Perhaps in the @params resourceBundleName of the getLogger(String, String) method. * @param resourceBundleName name of ResourceBundle to be used for localizing * messages for this logger. May be CODEnull/CODE if none of * the messages require localization. perhaps clarify that null only if the logger does not have a localization resource bundle set. Thanks Lance and Mandy! I have updated the webrev as per your suggestion: http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.01/ -- daniel thanks Mandy On 8/23/2013 7:41 AM, Daniel Fuchs wrote: Hi, Please find below a fix for: 8005899: Logger.getLogger(name, null) should not allow to reset a non-null resource bundle The issue here is that whereas in JDK 7, Logger.getLogger(foo, null); will throw an IllegalArgumentException if foo has a non null bundle name, JDK 8 will simply ignore the 'null' parameter and behave as Logger.getLogger(foo); The fix below restores the IllegalArgumentException in JDK 8. http://cr.openjdk.java.net/~dfuchs/webrev_8005899/webrev.00/ best regards, -- daniel Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com