JDK-8134502 : introduce abstraction for basic NodeVisitor usage
- Type: Enhancement
- Component: core-libs
- Sub-Component: jdk.nashorn
- Priority: P5
- Status: Resolved
- Resolution: Fixed
- Submitted: 2015-08-26
- Updated: 2016-01-14
- Resolved: 2015-09-28
The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
To download the current JDK release, click here.
Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
To download the current JDK release, click here.
| JDK 8 | JDK 9 |
|---|---|
8u72Fixed  |
9 b85Fixed |
Description
Throughout the Nashorn code base, there are many occurrences of anonymous inner classes (and non-anonymous classes) that extend NodeVisitor<LexicalContext>. To reduce noise, an abstract class BasicNodeVisitor that extends NodeVisitor<LexicalContext> should be introduced. If possible, common code used in all / many of the current occurrences should also be factored out to appropriate methods in the BasicNodeVisitor.