]> source.dussan.org Git - aspectj.git/commit
Store only weaved class names in the generatedClasses map
authorUri Simchoni <urisimchoni@gmail.com>
Mon, 5 Feb 2024 06:34:35 +0000 (08:34 +0200)
committerAlexander Kriegisch <Alexander@Kriegisch.name>
Thu, 8 Feb 2024 05:03:50 +0000 (12:03 +0700)
commit8a4aa038456b94762bb10b1b0e4b9935fd81c971
treef8c4fa72299228b8bd64b7d5ee01b3e3ef3e468c
parentae96aff0288c32951a84a5351bcd7b58ee40bea2
Store only weaved class names in the generatedClasses map

The generatedClasses map contained both keys of woven classes that had
generated classes associated with them, and the keys of the generated
classes themselves. It seems like this map is never consulted for the
generated class key - the generated class is generated from within the
context of woven class loading / retransformation, and thus no transform
event is generated for it by the JVM. Because of that, we do not need to
guard against re-weaving it. Other uses of generatedClasses map are for
full/empty tests, where the woven class key is sufficient. This change
simplifies deletion of a class because we do not have to look for its
associated generated classes.

Relates to #279.
loadtime/src/main/java/org/aspectj/weaver/loadtime/ClassLoaderWeavingAdaptor.java
weaver/src/main/java/org/aspectj/weaver/tools/WeavingAdaptor.java