RFR: 8305321: Remove unused exports in java.desktop [v3]

[Daniel Jeliński]

On Wed, 11 Oct 2023 16:38:16 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Daniel Jeliński has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'origin' into desktop-exports
>>  - Merge remote-tracking branch 'origin' into desktop-exports
>>  - Make J2dTraceInit static
>>  - Remove unnecessary exports
>
> src/java.desktop/share/native/libawt/java2d/SurfaceData.h line 557:
> 
>> 555:  */
>> 556: SurfaceDataOps *
>> 557: SurfaceData_GetOpsNoSetup(JNIEnv *env, jobject sData);
> 
> ~~It seems to me none of the functions in `SurfaceData.h` should be exported. They all end up in `awt.dll` / `libawt.so`. I can't see any of these functions are accessed from other DLLs.~~
> 
> ~~Would you like to create a separate task for removing exports from `SurfaceData` functions?~~
> 
> Some are used from `fontmanager.dll`. But not as many as we export.
> 
> Potentially all symbols exported from `awt.dll` except from `Java*` can be hidden…  
> I found that `Disposer_AddRecord` and `J2dTraceImpl` are used from `lcms.dll`.

Keep in mind that on Linux libawt is split into libawt, libawt_headless and libawt_xawt, and the export/import dependencies are different.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13261#discussion_r1355472800