[java] Update rule DontImportSun#3391
Conversation
Do not special case sun.misc.Signal anymore
Generated by 🚫 Danger |
Yes, it makes no sense to handle These classes were a big topic when Java 9 introduced the module system. You can still enable access to these classes I think, but they are not available by default anymore. |
Describe the PR
sun.misc.Signalis not ignored anymore. This struck me as very peculiar, since there's nothing special about Signal that makes it less unstable than egsun.misc.Unsafe... The way I see it, the goal of the rule is not to forbid using Unsafe or Signal or whatever, but to avoid doing it everywhere, and to confine those dependencies to small scopes, in which the rule can be suppressed. This applies to Signal as well and it weakens the rule to make an exception for it.Related issues
Ready?
./mvnw clean verifypasses (checked automatically by github actions)