RFR: 8318091: Remove empty initIDs functions

Phil Race prr at openjdk.org
Thu Oct 26 19:28:33 UTC 2023


On Thu, 26 Oct 2023 06:56:16 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> The removed functions are empty on all platforms.
> 
> This patch also removes calls to `Toolkit.loadLibraries();` in classes that no longer have any native methods. The call was needed to ensure that the native awt library is loaded.
> 
> Client libs tests passed.

Marked as reviewed by prr (Reviewer).

So this looks OK. The main thing I would worry about is that there isn't some code path which needs
the call to Toolkit.loadLibraries(), wasn't doing it itself, and gets by only because the libraries were loaded
by one of these calls being removed.

It is difficult to prove there is no such case. Tests may not be 100% coverage of all code paths.
Of course in such a case it is really a bug in that other code path, so if it shows up the fix would
likely be there rather than reverting some part of this change.

-------------

PR Review: https://git.openjdk.org/jdk/pull/16372#pullrequestreview-1700500301
PR Comment: https://git.openjdk.org/jdk/pull/16372#issuecomment-1781769779


More information about the client-libs-dev mailing list