Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support functions with a name for Java 11 & 12 #724

Closed
ptrthomas opened this issue Apr 4, 2019 · 2 comments
Closed

Support functions with a name for Java 11 & 12 #724

ptrthomas opened this issue Apr 4, 2019 · 2 comments
Assignees
@ptrthomas
Labels
bug fixed
Milestone
0.9.3

Comments

@ptrthomas
Copy link
Member

In Nashorn Java 11 for some reason functions with a name are considered invalid.

Refer this Stack Overflow thread: https://stackoverflow.com/a/54554226/143475

We have a fix for this that was developed for the graal branch: 7cc24d0#diff-9e2ac7fbf82c12b0fe45b5b64b3711c2R206
@ptrthomas ptrthomas added the bug label Apr 4, 2019
@ptrthomas ptrthomas self-assigned this Apr 4, 2019
@ptrthomas ptrthomas mentioned this issue Apr 4, 2019
Closed
5 tasks
ptrthomas added a commit that referenced this issue Apr 4, 2019
@ptrthomas
remove js function name if present #724
5916e69
@ptrthomas ptrthomas added the fixed label Apr 4, 2019
@ptrthomas
Copy link
Member Author

@LeJeanbono would you be able to build from source and confirm that this is fixed ?

@MercierAdrien MercierAdrien mentioned this issue Apr 18, 2019
Closed
@ptrthomas ptrthomas added this to the 0.9.3 milestone Apr 23, 2019
@ptrthomas ptrthomas mentioned this issue May 15, 2019
Closed
@ptrthomas ptrthomas changed the title Support functions with a name for Java 11 Support functions with a name for Java 11 & 12 May 16, 2019
@ptrthomas
Copy link
Member Author

released 0.9.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ptrthomas ptrthomas

Labels
bug fixed
Projects
None yet
Milestone
0.9.3
Development

No branches or pull requests
1 participant
@ptrthomas