-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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.
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
Make :javap
in REPL work on JDK 11+
#8486
Conversation
src/repl-frontend/scala/tools/nsc/interpreter/shell/JavapClass.scala
Outdated
Show resolved
Hide resolved
2881c5a
to
0de71b9
Compare
|
||
def compilerClasspath: Seq[URL] | ||
|
||
def outputDir: AbstractFile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suboptimal for now, bytecode printing should be a service.
Requested retronym who requested a less budget-busting version. The LOC delta is a wash after refactor. I've already forgotten what it does. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bravely straddling the JDK version chasm.
Later, we could consider making the code cleaner by making our build messier letting sub-projects be built with particular JDKs (or JDK.latest with particular -release
arguments) and packaging the result up in a multi-release JAR. This could be useful in other spots of the library.
and another "Forward port javap support" ticket: lampepfl/dotty-feature-requests#80 |
:javap
in REPL work on JDK 11+
No description provided.