On Tue, 23 Sep 2025 13:19:48 GMT, Jayathirth D V <[email protected]> wrote:
> Regarding not using PlatformUtil methods : If i import PlatformUtil class in > these tests i get `package com.sun.javafx is declared in module javafx.base, > which does not export it to the unnamed module`. I tried adding > `--add-exports=javafx.base/com.sun.javafx.PlatformUtil=ALL-UNNAMED` in > compiler args of swt job, that didn't work out. Also i was not sure whether > we need to export this javafx.base class or not. So i went ahead and added > utility code in test. > > Since we will be disabling this test completely, i will go ahead and remove > all PlatformUtil related code. As you say, this is no longer needed for this PR. If something like this arises in the future, the way to add the needed export is to create a `modules/javafx.swt/src/test/addExports` file (the system tests and most of the modules already have one, but `javafx.swt` does not) and the add the following line: --add-exports javafx.base/com.sun.javafx=ALL-UNNAMED (note that exports are done at the package level not per class) ------------- PR Comment: https://git.openjdk.org/jfx/pull/1917#issuecomment-3324590689
