RFR: 8220483: Calendar.setTime(Date date) throws NPE with Date date = null

Joe Wang joehw at openjdk.java.net
Mon Sep 14 22:55:16 UTC 2020


On Mon, 14 Sep 2020 22:41:55 GMT, Lance Andersen <lancea at openjdk.org> wrote:

>> Hi,
>> 
>> Please review this simple doc fix.
>
> Hi Naoto,
> 
> The change looks fine.  I might have created a CSR just to track the doc change.

The change looks fine. However, are other methods in the same situation, e.g. void setTimeZone​(TimeZone value),
getInstance​(Locale aLocale), etc.?  Or, would it better to add a general statement about passing in null values?

-------------

PR: https://git.openjdk.java.net/jdk/pull/159


More information about the core-libs-dev mailing list