Classes generated by deftype and defrecord don't play nice with .getPackage
Description
Classes generated loaded by DynamicClassLoader return nil for .getPackage. Tools like CIDER and vim-fireplace are relying on this information to implement things like completion hints.
Proposed: During DynamicClassLoader.defineClass(), invoke definePackage() on the class being defined (similar to what URLClassLoader does).
Patch: clj-1550-v4.patch
Screened by: Alex Miller
Environment
Activity
That's the current path...
This is a bad patch that slipped through. The motivation linked above shows an nREPL excerpt that attempts to symbolize the package, basically:
. The original code can derive the package name in the same way the JVM grabs it, from the binary name. (Take the prefix up until the last period.)
The fix also introduces a deprecated call, and more work during classloading (which is critical path).
As an aside, JDK9+ automatically defines a package upon class definition, derived from the binary name:
I think this should get reverted, and let user space calculate the package from the binary name.
> The original code can derive the package name in the same way the JVM grabs it, from the binary name. (Take the prefix up until the last period.)
I don't completely understand the workaround you're proposing. Can you elaborate on this?
> I think this should get reverted, and let user space calculate the package from the binary name.
If that's working on newer JDKs with no changes needed in the user space and in Clojure, that's fine by me. By now I had lost faith that was going to be fixed in Clojure anyways.
Took me a while to track it down, but instead of asking for the package name then grabbing the symbol, do this:
Patch reverted
Assignee
Reporter
Labels
Approval
Patch
Affects versions
Priority
