blackdrag commented on code in PR #2786:
URL: https://github.com/apache/groovy/pull/2786#discussion_r3790751828
##########
src/main/java/org/apache/groovy/runtime/indy/IndyInvalidation.java:
##########
@@ -180,15 +180,23 @@ public static void invalidateUnscoped() {
}
/**
- * Detaches and invalidates every live class-level MetaClass / pending
domain
- * for loaded types. Shared implementation for all process-wide entries.
+ * Detaches and invalidates every live MetaClass / pending domain via the
+ * live-SwitchPoint registry — O(live domains), replacing the former walk
+ * over {@link ClassInfo#getAllClassInfo()} which cost O(loaded classes)
+ * twice per category {@code use} block (GROOVY-12259). Shared
+ * implementation for all process-wide entries.
+ * <p>
Review Comment:
As I said to Daniel in the other PR, this is very much AI generated work
report content. I don´t think it should be in the javadoc. This PR changes what
the class does, it is not the place of the javadoc to mention what it did
before, unless for compatibility, but compatibility is not the focus here.
Everything after live domains and before Shared should be removed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
