Re: RFR 8160036: Java API doc for method minusMonths in LocalDateTime class needs correction

2016-11-23 Thread Stephen Colebourne 
+1
Stephen

On 7 November 2016 at 21:23, Roger Riggs  wrote:
> Looks good,
>
> Thanks, Roger
>
>
>
> On 11/7/2016 7:27 AM, Bhanu Gopularam wrote:
>>
>> Thanks Nadeesh. Here is the updated webrev:
>>
>> http://cr.openjdk.java.net/~bgopularam/8160036/webrev.01
>>
>> Thanks,
>> Bhanu
>>
>> -Original Message-
>> From: nadeesh tv
>> Sent: Monday, November 07, 2016 5:38 PM
>> To: core-libs-dev@openjdk.java.net
>> Subject: Re: RFR 8160036: Java API doc for method minusMonths in
>> LocalDateTime class needs correction
>>
>> Hi Bhanu,
>> Same issues with OffsetDateTime minusWeeks and minusDays
>>
>> Thanks and Regards,
>> Nadeesh
>>
>> On 11/7/2016 5:31 PM, Bhanu Gopularam wrote:
>>>
>>> Hi all,
>>>
>>> Could you please review fix for following issue?
>>>
>>> Bug id: https://bugs.openjdk.java.net/browse/JDK-8160036
>>>
>>> Solution: Corrected documentation for couple of methods in LocalDateTime
>>> and OffsetDateTime classes
>>>
>>> Webrev: http://cr.openjdk.java.net/~bgopularam/8160036/webrev.00
>>>
>>> Thanks,
>>> Bhanu
>
>

Re: RFR 8160036: Java API doc for method minusMonths in LocalDateTime class needs correction

2016-11-07 Thread Roger Riggs 

Looks good,

Thanks, Roger


On 11/7/2016 7:27 AM, Bhanu Gopularam wrote:

Thanks Nadeesh. Here is the updated webrev:

http://cr.openjdk.java.net/~bgopularam/8160036/webrev.01

Thanks,
Bhanu

-Original Message-
From: nadeesh tv
Sent: Monday, November 07, 2016 5:38 PM
To: core-libs-dev@openjdk.java.net
Subject: Re: RFR 8160036: Java API doc for method minusMonths in LocalDateTime 
class needs correction

Hi Bhanu,
Same issues with OffsetDateTime minusWeeks and minusDays

Thanks and Regards,
Nadeesh

On 11/7/2016 5:31 PM, Bhanu Gopularam wrote:

Hi all,

Could you please review fix for following issue?

Bug id: https://bugs.openjdk.java.net/browse/JDK-8160036

Solution: Corrected documentation for couple of methods in LocalDateTime and 
OffsetDateTime classes

Webrev: http://cr.openjdk.java.net/~bgopularam/8160036/webrev.00

Thanks,
Bhanu

RE: RFR 8160036: Java API doc for method minusMonths in LocalDateTime class needs correction

2016-11-07 Thread Bhanu Gopularam 
Thanks Nadeesh. Here is the updated webrev:

http://cr.openjdk.java.net/~bgopularam/8160036/webrev.01

Thanks,
Bhanu

-Original Message-
From: nadeesh tv 
Sent: Monday, November 07, 2016 5:38 PM
To: core-libs-dev@openjdk.java.net
Subject: Re: RFR 8160036: Java API doc for method minusMonths in LocalDateTime 
class needs correction

Hi Bhanu,
Same issues with OffsetDateTime minusWeeks and minusDays

Thanks and Regards,
Nadeesh

On 11/7/2016 5:31 PM, Bhanu Gopularam wrote:
> Hi all,
>
> Could you please review fix for following issue?
>
> Bug id: https://bugs.openjdk.java.net/browse/JDK-8160036
>
> Solution: Corrected documentation for couple of methods in LocalDateTime and 
> OffsetDateTime classes
>
> Webrev: http://cr.openjdk.java.net/~bgopularam/8160036/webrev.00
>
> Thanks,
> Bhanu

-- 
Thanks and Regards,
Nadeesh TV

Re: RFR 8160036: Java API doc for method minusMonths in LocalDateTime class needs correction

2016-11-07 Thread nadeesh tv 

Hi Bhanu,
Same issues with OffsetDateTime minusWeeks and minusDays

Thanks and Regards,
Nadeesh

On 11/7/2016 5:31 PM, Bhanu Gopularam wrote:

Hi all,

Could you please review fix for following issue?

Bug id: https://bugs.openjdk.java.net/browse/JDK-8160036

Solution: Corrected documentation for couple of methods in LocalDateTime and 
OffsetDateTime classes

Webrev: http://cr.openjdk.java.net/~bgopularam/8160036/webrev.00

Thanks,
Bhanu


--
Thanks and Regards,
Nadeesh TV

RFR 8160036: Java API doc for method minusMonths in LocalDateTime class needs correction

2016-11-07 Thread Bhanu Gopularam 
Hi all,

Could you please review fix for following issue?

Bug id: https://bugs.openjdk.java.net/browse/JDK-8160036

Solution: Corrected documentation for couple of methods in LocalDateTime and 
OffsetDateTime classes

Webrev: http://cr.openjdk.java.net/~bgopularam/8160036/webrev.00

Thanks,
Bhanu