RFR JDK-8133079 LocalDate/LocalTime ofInstant()

[Roger Riggs]

Hi Stephen,

Yes, things that logically go together can be in the same bug/issue and 
be handled with a single
review cycle to keep the overhead down.

Thanks, Roger


On 9/10/2015 10:08 AM, Stephen Colebourne wrote:
> Shall I just open a number of these with things to be changed, so you
> can deal with them in a batch then?
> Stephen
>
>
> On 10 September 2015 at 14:42, Roger Riggs <Roger.Riggs at oracle.com> wrote:
>> Hi Stephen,
>>
>> I can take it up but probably won't get to it until after JavaOne. (Early
>> Nov)
>>
>> Roger
>>
>>
>>
>> On 9/8/2015 2:53 PM, Stephen Colebourne wrote:
>>> Anyone like to take this on please?
>>> Stephen
>>> On 28 Aug 2015 00:07, "Stephen Colebourne" <scolebourne at joda.org> wrote:
>>>
>>>> External question sites indicate that users have difficulty converting
>>>> between java.util.Date and LocalDate, and also between Instant and
>>>> LocalDate/LocalTime. This user difficulty can be resolved with some
>>>> additional methods.
>>>>
>>>> Currently, the following methods exist:
>>>>
>>>>    ZonedDateTime.ofInstant(Instant, ZoneId);
>>>>    OffsetDateTime.ofInstant(Instant, ZoneId);
>>>>    LocalDateTime.ofInstant(Instant, ZoneId);
>>>>    OffsetTime.ofInstant(Instant, ZoneId);
>>>>
>>>> This enhancement proposes to add the same method to LocalDate and
>>>> LocalTime. It can reasonably be argued this was a simple oversight in
>>>> the original development.
>>>>
>>>> Diff attached to the JIRA:
>>>>
>>>> https://bugs.openjdk.java.net/browse/JDK-8133079
>>>>
>>>> Would also appreciate a reviewer to ensure that the code and tests
>>>> pass OK (as opposed to my best efforts on a very dubious
>>>> Windows/Eclipse/JDK8 setup)
>>>>
>>>> Stephen
>>>>