JDK11: Add support for the CONSTANT_Dynamic 17 constant introduced in Java 11

[richbolen]

Java 11 introduced a new constant called Constant_Dynamic (defined here https://docs.oracle.com/javase/specs/jvms/se11/html/jvms-4.html#jvms-4.4-210). When running junit tests using powermock on the Java 11 I get this error: invalid constant type: 17 at 130. Support for this constant needs to be added to the ConstPool class.

[chibash]

OK, I'll work on this.

[richbolen]

I've got it implemented locally. I can submit a PR. I'm still working on the unit test but I could submit what I have.

[richbolen]

I don't have permission to submit a PR but would be happy to if you are interested.

[chibash]

I think anyone can submit a pull request. They are not allowed to merge it, though.

[richbolen]

Yes. It was a issue with the macos and SSH keys. Created PR #272

[camron-johnson]

Is there any update on the addition of Constant_Dynamic to Javassist?

[Bophie]

发自我的iPhone

…

------------------ Original ------------------ From: richbolen <notifications@github.com> Date: Tue,Aug 20,2019 11:02 PM To: jboss-javassist/javassist <javassist@noreply.github.com> Cc: Subscribed <subscribed@noreply.github.com> Subject: Re: [jboss-javassist/javassist] JDK11: Add support for the CONSTANT_Dynamic 17 constant introduced in Java 11 (#270) Yes. It was a issue with the macos and SSH keys. Created PR #272 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread. 从QQ邮箱发来的超大附件 试音.zip (8.6M, 无限期) 进入下载页面：https://wx.mail.qq.com/ftn/download?func=3&key=9a9c6363d1f56e1da8ef1663316130315ffcba6333613031404c5d0001035257525a15565059021c510208541e5803055d4e0e570007000707510e5a52563a318dccad8aacd21e4b0c132cc6b769a5820e5d8f520a8eecfa3f80c7a1bd6e69&code=ec8c3a01&k=9a9c6363d1f56e1da8ef1663316130315ffcba6333613031404c5d0001035257525a15565059021c510208541e5803055d4e0e570007000707510e5a52563a318dccad8aacd21e4b0c132cc6b769a5820e5d8f520a8eecfa3f80c7a1bd6e69

[valfirst]

@chibash are there any plans to release a new version with this feature?

[chibash]

I've just released. It'll be available from maven within several days.

[atulrulers]

is it resolved ? Any ETA ?

[valfirst]

yes, it's fixed (I believe someone from maintainers should close it)