Fix missing unsafe available check#6920
Conversation
trask
left a comment
There was a problem hiding this comment.
is it worth (possible?) adding something in CI that runs the tests without unsafe?
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6920 +/- ##
============================================
- Coverage 90.10% 90.09% -0.02%
Complexity 6594 6594
============================================
Files 729 729
Lines 19828 19828
Branches 1950 1951 +1
============================================
- Hits 17867 17865 -2
Misses 1369 1369
- Partials 592 594 +2 ☔ View full report in Codecov by Sentry. |
added a test that uses a custom class loader to hide sun.misc.Unsafe from the tested code |
|
Thanks for the fix! |
|
Can confirm this fixes the issue. 👍 |
ah I see the problem with using something like
|
Resolves #6913