On 1/31/19, 11:30 AM, Brian Burkhalter wrote:
Hi Joe,

It looks like lines 65-66 of src/java.xml/share/classes/com/sun/org/apache/xpath/internal/functions/FuncDoclocation.java can go away as well:

//        int owner = dtm.getDocument();
// fileLocation = xctxt.getSourceTreeManager().findURIFromDoc(owner);

Done.

Updated the webrev with the addition of FuncDoclocation:
http://cr.openjdk.java.net/~joehw/jdk13/8186321/webrev/

Thanks,
Joe


Thanks,

Brian

On Jan 31, 2019, at 11:18 AM, Joe Wang <huizhe.w...@oracle.com <mailto:huizhe.w...@oracle.com>> wrote:

Please review a cleanup of an unused class. All core and xml tests passed.

JBS:https://bugs.openjdk.java.net/browse/JDK-8186321
webrevs:http://cr.openjdk.java.net/~joehw/jdk13/8186321/webrev/ <http://cr.openjdk.java.net/%7Ejoehw/jdk13/8186321/webrev/>

Reply via email to