Thanks Lance! I somehow copied the wrong link. Although they point to the same document, the shorter version (without the prefix and date) is better. Fixed now in the webrev:
webrev: http://cr.openjdk.java.net/~joehw/jdk12/8212871/webrev/

Thanks,
Joe

On 10/31/18, 5:14 PM, Lance Andersen wrote:
BTW

This seems to only be for:

--- old/src/jdk.xml.dom/share/classes/org/w3c/dom/xpath/XPathNamespace.java     
2018-10-31 16:49:00.265127157 -0700
+++ new/src/jdk.xml.dom/share/classes/org/w3c/dom/xpath/XPathNamespace.java     
2018-10-31 16:48:59.861089532 -0700
@@ -74,7 +74,7 @@
   * future versions of the XPath specification, the definition of a namespace
   * node may be changed incomatibly, in which case incompatible changes to
   * field values may be required to implement versions beyond XPath 1.0.
- *<p>See also the<a 
href='http://www.w3.org/2002/08/WD-DOM-Level-3-XPath-20020820'>Document Object Model (DOM) 
Level 3 XPath Specification</a>.
+ *<p>See also the<a 
href='https://www.w3.org/TR/2004/NOTE-DOM-Level-3-XPath-20040226/'>Document Object Model (DOM) 
Level 3 XPath Specification</a>.
   */

On Oct 31, 2018, at 8:09 PM, Lance Andersen <lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>> wrote:

Hi Joe,

I see the update as

https://www.w3.org/TR/DOM-Level-3-XPath/
and

https://www.w3.org/TR/2004/NOTE-DOM-Level-3-XPath-20040226/ <https://www.w3.org/TR/2004/NOTE-DOM-Level-3-XPath-20040226/>

Was that intentional?


On Oct 31, 2018, at 8:02 PM, Joe Wang <huizhe.w...@oracle.com> wrote:

Hi,

Please review a fix for the broken links, replacing:
http://www.w3.org/2002/08/WD-DOM-Level-3-XPath-20020820
with:
https://www.w3.org/TR/DOM-Level-3-XPath/

The former on which the jdk.xml.dom package was based is no longer publicly available. The later is current. The only difference between the two was noted in the package info ( src/jdk.xml.dom/share/classes/org/w3c/dom/xpath/package-info.java), and also in XPathException.java in this webrev.

JBS: https://bugs.openjdk.java.net/browse/JDK-8212871
webrev: http://cr.openjdk.java.net/~joehw/jdk12/8212871/webrev/

Thanks,
Joe


<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>




<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>



Reply via email to