Thanks Roger, made the suggested changes.
Thanks,
Pallavi Sonal
From: Roger Riggs
Sent: Monday, October 01, 2018 9:29 PM
To: Pallavi Sonal ; Naoto Sato
; core-libs-dev
Subject: Re: RFR: JDK-8166138 - DateTimeFormatter.ISO_INSTANT should handle
offsets
Hi,
The checkbox
Hi Naoto,
Here is the CSR for review :
https://bugs.openjdk.java.net/browse/JDK-8211316
Thanks,
Pallavi Sonal
-Original Message-
From: Naoto Sato
Sent: Friday, September 28, 2018 9:38 PM
To: Pallavi Sonal ; Roger Riggs
; core-libs-dev
Subject: Re: RFR: JDK-8166138
Thanks Roger. I will update it before the commit.
-Original Message-
From: Roger Riggs
Sent: Friday, September 28, 2018 6:15 PM
To: Pallavi Sonal ; core-libs-dev
Subject: Re: RFR: JDK-8166138 - DateTimeFormatter.ISO_INSTANT should handle
offsets
Hi Pallavi,
Looks fine,
But can you
Thanks Roger and Stephen. Here is the updated webrev with the suggested changes:
http://cr.openjdk.java.net/~rpatil/8166138/webrev.04/
Thanks,
Pallavi Sonal
-Original Message-
From: Roger Riggs
Sent: Friday, September 28, 2018 12:51 AM
To: core-libs-dev
Subject: Re: RFR: JDK-8166138
Thanks for the clarification. Here is the updated webrev for review :
http://cr.openjdk.java.net/~rpatil/8166138/webrev.03/
Thanks,
Pallavi Sonal
-Original Message-
From: Stephen Colebourne
Sent: Tuesday, September 25, 2018 4:39 PM
To: core-libs-dev
Subject: Re: RFR: JDK-8166138
as necessary.
thanks
Stephen
On Fri, 21 Sep 2018 at 13:26, Pallavi Sonal wrote:
>
> Thank you Stephen for your inputs. Based on that, here is the updated webrev
> for review :
> http://cr.openjdk.java.net/~rpatil/8166138/webrev.02/
>
> Thanks,
> Pallavi Sonal.
>
> -O
Thank you Stephen for your inputs. Based on that, here is the updated webrev
for review :
http://cr.openjdk.java.net/~rpatil/8166138/webrev.02/
Thanks,
Pallavi Sonal.
-Original Message-
From: Stephen Colebourne
Sent: Thursday, September 20, 2018 6:38 PM
To: core-libs-dev
Subject
Thanks Roger , Naoto and Stephen for the review and valuable inputs.
Here is the updated webrev for review :
http://cr.openjdk.java.net/~rpatil/8166138/webrev.01/
Thanks,
Pallavi Sonal
-Original Message-
From: Stephen Colebourne
Sent: Thursday, September 20, 2018 2:50 AM
To: core
, "+", or "+00" will not be accepted in the input string to be
parsed.
[1] https://en.wikipedia.org/wiki/ISO_8601
Thanks,
Pallavi Sonal
d the
adjacent value parsing only happens with another parser for the consecutive
non-padded fields.
Should the PadPrinterDecoratorParsers be allowed to participate in adjacent
value parsing and should this affect both padded followed by non-padded fields
as well as non-padded followed
tException is thrown.
The fix has been done to check such cases where the previous parserprinter is
PadPrinterDecoratorParser and use the new NumberPrinterParser instead for
parsing the next value.
All the tier1 and tier2 Mach 5 tests have passed.
Thanks,
Pallavi Sonal
of http.nonProxyHosts , the code works as
expected and DIRECT connection is used for them. But in scenarios, where there
is a wildcard both at the beginning and end of the hostname, proxy is used
instead of DIRECT connection.
All the tier1 and tier2 Mach 5 tests have passed.
Thanks,
Pallavi
12 matches
Mail list logo