On Wed, 30 Oct 2024 18:16:26 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

Looks good. I think the changes in `PlatformImpl` are OK.

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

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1611#pullrequestreview-2405954457

Reply via email to