RFR (JDK 13/java.xml) 8186321: Cleanup: SourceTreeManager not used

Joe Wang huizhe.wang at oracle.com
Thu Jan 31 20:01:41 UTC 2019


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.wang at oracle.com 
>> <mailto:huizhe.wang at 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/>
>


More information about the core-libs-dev mailing list