On Wed, 30 Oct 2024 18:13:48 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> Removes doPrivileged in the following module:
>> 
>> - javafx.swt
>> 
>> See JDK-8342441 for details.
>> 
>> As a helpful hint for reviewers, I recommend reviewing this using the "Hide 
>> whitespace" option.
>
> Andy Goryachev 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:
> 
>  - review comments
>  - Merge remote-tracking branch 'origin/master' into 8342914.do.privileged.swt
>  - Merge remote-tracking branch 'origin/master' into 8342914.do.privileged.swt
>  - 8342914: Remove calls to doPrivileged in swt

modules/javafx.graphics/src/main/java/com/sun/javafx/application/PlatformImpl.java
 line 361:

> 359:     // FXCanvas-specific initialization
> 360:     private static void initFXCanvas() {
> 361:         // Verify that we have the appropriate permission

this might create a (trivial) merge conflict

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1611#discussion_r1823161289

Reply via email to