Re: RFR 8246034: Remove java.base/share/classes/jdk/internal/jrtfs/jrtfsviewer.js and java.base/share/classes/jdk/internal/jrtfs/jrtls.js

2020-05-28 Thread sundararajan . athijegannathan 

Yes, checked. No config file refers to these .js files.

-Sundar

On 28/05/20 12:28 pm, Alan Bateman wrote:

On 28/05/2020 07:14, sundararajan.athijegannat...@oracle.com wrote:

Please review.

Bug: https://bugs.openjdk.java.net/browse/JDK-8246034

Webrev: http://cr.openjdk.java.net/~sundar/8246034/webrev.00/
Have you checked if there are any make file configured to copy .js 
files? Otherwise looks good.


-Alan

Re: RFR 8246034: Remove java.base/share/classes/jdk/internal/jrtfs/jrtfsviewer.js and java.base/share/classes/jdk/internal/jrtfs/jrtls.js

2020-05-28 Thread Alan Bateman 

On 28/05/2020 07:14, sundararajan.athijegannat...@oracle.com wrote:

Please review.

Bug: https://bugs.openjdk.java.net/browse/JDK-8246034

Webrev: http://cr.openjdk.java.net/~sundar/8246034/webrev.00/
Have you checked if there are any make file configured to copy .js 
files? Otherwise looks good.


-Alan

Re: RFR 8246034: Remove java.base/share/classes/jdk/internal/jrtfs/jrtfsviewer.js and java.base/share/classes/jdk/internal/jrtfs/jrtls.js

2020-05-28 Thread James Laskey 
+1



> On May 28, 2020, at 3:16 AM, sundararajan.athijegannat...@oracle.com wrote:
> 
> Please review.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8246034
> 
> Webrev: http://cr.openjdk.java.net/~sundar/8246034/webrev.00/
> 
> Thanks
> 
> -Sundar
>

RFR 8246034: Remove java.base/share/classes/jdk/internal/jrtfs/jrtfsviewer.js and java.base/share/classes/jdk/internal/jrtfs/jrtls.js

2020-05-28 Thread sundararajan . athijegannathan 

Please review.

Bug: https://bugs.openjdk.java.net/browse/JDK-8246034

Webrev: http://cr.openjdk.java.net/~sundar/8246034/webrev.00/

Thanks

-Sundar