Thanks Lance!
On 5/17/2016 11:26 AM, Lance Andersen wrote:
Hi Joe
It looks OK. The formatting is a bit off where you encased existing
code with new if statements so I would consider fixing that before
you push
It seems it has something to do with whitespaces. I re-generated the
webrev. Also, I was missing a new file "uri.xml" in the webrev.
http://cr.openjdk.java.net/~joehw/jdk9/8156845/webrev/
Best,
Joe
Best
Lance
On May 17, 2016, at 12:42 PM, huizhe wang <huizhe.w...@oracle.com
<mailto:huizhe.w...@oracle.com>> wrote:
Hi,
This patch fixes a missing feature in CatalogUriResolver where an uri
entry can be referenced by a publicid (urn:publicid), similar to that
of the public entry. The fix then is basically sharing the process
for a public entry.
JBS: https://bugs.openjdk.java.net/browse/JDK-8156845
webrev: http://cr.openjdk.java.net/~joehw/jdk9/8156845/webrev/
<http://cr.openjdk.java.net/%7Ejoehw/jdk9/8156845/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>
