[8u-dev] Request for Approval (JAXP): 8028363: XmlGregorianCalendarImpl.getTimeZone() bug when offset is less than 10 minutes

[Rob McKenna]

Approved

	-Rob

On 22/07/16 01:40, Svetlana Nikandrova wrote:
> Hello,
> 
> please approve integration of the fix to jdk8u-dev.
> Webrev (2 parts):
> http://cr.openjdk.java.net/~snikandrova/8028363/jaxp/webrev.00/
> http://cr.openjdk.java.net/~snikandrova/8028363/jdk/webrev.00/
> JBS:
> https://bugs.openjdk.java.net/browse/JDK-8028363
> Review thread:
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-July/042502.html
> 
> Thank you,
> Svetlana
> 
> On 21.07.2016 20:40, Seán Coffey wrote:
> >Looks fine to me also Svetlana.
> >
> >Regards,
> >Sean.
> >
> >On 21/07/2016 13:13, Svetlana Nikandrova wrote:
> >>Little reminder.
> >>
> >>On 19.07.2016 19:29, Svetlana Nikandrova wrote:
> >>>Hello,
> >>>
> >>>please review this fix for 8u-dev. For jdk9 problem has been addressed
> >>>in JEP 255 <https://bugs.openjdk.java.net/browse/JDK-8044086> (Xerces
> >>>update).
> >>>Please note that changeset contains of 2 parts:
> >>>XMLGregorianCalendarImpl fix in jaxp ws:
> >>>http://cr.openjdk.java.net/~snikandrova/8028363/jaxp/webrev.00/
> >>><http://cr.openjdk.java.net/%7Esnikandrova/8028363/jaxp/webrev.00/>
> >>>Regression test in jdk ws:
> >>>http://cr.openjdk.java.net/~snikandrova/8028363/jdk/webrev.00/
> >>><http://cr.openjdk.java.net/%7Esnikandrova/8028363/jdk/webrev.00/>
> >>>
> >>>JBS:
> >>>https://bugs.openjdk.java.net/browse/JDK-8028363
> >>>
> >>>Description:
> >>>While concatenating timezone representations string for
> >>>TimeZone.getTimeZone()  XmlGregorianCalendarImpl missed prior "0" in
> >>>minutes part if minutes value is less then 10 (like "0" in 11:05).
> >>>
> >>>Thank you,
> >>>Svetlana
> >>>
> >>
> >
>