RFR (JDK10/JAXP Doc-only) 8189704: broken links in the javax/xml/namespace package

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR (JDK10/JAXP Doc-only) 8189704: broken links in the javax/xml/namespace package

huizhe wang
Hi,

Please review a doc-only fix for two namespace classes. The broken links
to the Errata are removed since the reference(s) to the Errata is part
of the Namespaces in XML specification itself. There is no need for the
extra link(s).  Also, for NamespaceContext, the link to "Namespaces in
XML" shall be to the specification, not the QName section, the fragment
portion of the link, e.g. "#ns-qualnames" therefore is also removed.

JBS: https://bugs.openjdk.java.net/browse/JDK-8189704
webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8189704/webrev/

Thanks,
Joe
Reply | Threaded
Open this post in threaded view
|

Re: RFR (JDK10/JAXP Doc-only) 8189704: broken links in the javax/xml/namespace package

huizhe wang
This change also fix:
JBS: https://bugs.openjdk.java.net/browse/JDK-8184046

that contains the same broken link to "Namespaces in XML Errata". As
Daniel indicated, the correct link was with a ".html" suffix. Since the
content in this Errata has already been incorporated in the current
specification that the link to "Namespaces in XML" points to, there's no
more reason to link to the Errata.

Thanks,
Joe

On 1/3/2018 3:32 PM, huizhe wang wrote:

> Hi,
>
> Please review a doc-only fix for two namespace classes. The broken
> links to the Errata are removed since the reference(s) to the Errata
> is part of the Namespaces in XML specification itself. There is no
> need for the extra link(s).  Also, for NamespaceContext, the link to
> "Namespaces in XML" shall be to the specification, not the QName
> section, the fragment portion of the link, e.g. "#ns-qualnames"
> therefore is also removed.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8189704
> webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8189704/webrev/
>
> Thanks,
> Joe

Reply | Threaded
Open this post in threaded view
|

Re: RFR (JDK10/JAXP Doc-only) 8189704: broken links in the javax/xml/namespace package

joe darcy
In reply to this post by huizhe wang
Looks fine Joe; cheers,

-Joe

On 1/3/2018 3:32 PM, huizhe wang wrote:

> Hi,
>
> Please review a doc-only fix for two namespace classes. The broken
> links to the Errata are removed since the reference(s) to the Errata
> is part of the Namespaces in XML specification itself. There is no
> need for the extra link(s).  Also, for NamespaceContext, the link to
> "Namespaces in XML" shall be to the specification, not the QName
> section, the fragment portion of the link, e.g. "#ns-qualnames"
> therefore is also removed.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8189704
> webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8189704/webrev/
>
> Thanks,
> Joe

Reply | Threaded
Open this post in threaded view
|

Re: RFR (JDK10/JAXP Doc-only) 8189704: broken links in the javax/xml/namespace package

huizhe wang
Thanks Joe!

Cheers,
Joe

On 1/3/2018 4:47 PM, Joseph D. Darcy wrote:

> Looks fine Joe; cheers,
>
> -Joe
>
> On 1/3/2018 3:32 PM, huizhe wang wrote:
>> Hi,
>>
>> Please review a doc-only fix for two namespace classes. The broken
>> links to the Errata are removed since the reference(s) to the Errata
>> is part of the Namespaces in XML specification itself. There is no
>> need for the extra link(s).  Also, for NamespaceContext, the link to
>> "Namespaces in XML" shall be to the specification, not the QName
>> section, the fragment portion of the link, e.g. "#ns-qualnames"
>> therefore is also removed.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8189704
>> webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8189704/webrev/
>>
>> Thanks,
>> Joe
>