Replies: 1 comment
-
hey, I was just thinking about this yesterday 😁 I think we could add a test flag that triggers a failure if we aren't using the good field-based implementation (using the fallback weak map), and then if we find there are a few classes, we can allow only those specific classes to use the fallback weak map under the test flag. I'm not sure if this would have been possible before #1643, because the tests relied on retransforming lots of classes, but I think it should be possible now. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Do we have a way to confirm (ideally integration test) which classes are correctly being instrumented with a field by FieldBackedProvider? It's mostly in the context of #2601, by using more stuff in the bootstrap classloader I think there's always a chance of accidentally classloading something too early. How do we confirm we don't introduce such a regression?
Beta Was this translation helpful? Give feedback.
All reactions