Details
-
Bug
-
Resolution: Fixed
-
P2
-
9
-
b89
-
generic
-
generic
Backports
| Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
|---|---|---|---|---|---|---|
| JDK-8142851 | emb-9 | Attila Szegedi | P2 | Resolved | Fixed | team |
Description
After JDK-8139931, the test/script/basic/JDK-8023630.js fails.
The reason is a fairly late change to 8139931 where all uses of the (in the end removed) CompositeOperation.contains() method were replaced with NashornCallSiteDescriptor.getFirstStandardOperation(). Unfortunately, uses within JavaSuperAdapterLinker and ReflectionCheckLinker should have been left as-is using CompositeOperation.contains().
Solution is to reinstantiate CompositeOperation.contains() and make sure JavaSuperAdapterLinker and ReflectionCheckLinker use them accordingly.
The reason is a fairly late change to 8139931 where all uses of the (in the end removed) CompositeOperation.contains() method were replaced with NashornCallSiteDescriptor.getFirstStandardOperation(). Unfortunately, uses within JavaSuperAdapterLinker and ReflectionCheckLinker should have been left as-is using CompositeOperation.contains().
Solution is to reinstantiate CompositeOperation.contains() and make sure JavaSuperAdapterLinker and ReflectionCheckLinker use them accordingly.
Attachments
Issue Links
- backported by
-
JDK-8142851 Restore use of CompositeOperation.contains where it is needed
-
- Resolved
-
- relates to
-
JDK-8139931 Introduce Operation objects in Dynalink instead of string encoding
-
- Resolved
-