Re: RFR: 8344799: Remove permissions checks from java.awt.Desktop [v2]

2024-11-21 Thread Alexander Zvegintsev
On Fri, 22 Nov 2024 00:01:32 GMT, Phil Race wrote: >> Clean up permission checks in java.awt.Desktop class >> Also remove no longer needed exports from java.base to java.desktop > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: RFR: 8344799: Remove permissions checks from java.awt.Desktop

2024-11-21 Thread Alexander Zvegintsev
On Thu, 21 Nov 2024 23:58:05 GMT, Phil Race wrote: > Did you mean to mention that in this PR ? Not sure how it came up here. Yes, I saw that the `sun.security.action` is no longer exported to `java.desktop`, and just wanted to check that it is no longer used in there(even in comments). --

Re: RFR: 8344799: Remove permissions checks from java.awt.Desktop

2024-11-21 Thread Alexander Zvegintsev
On Thu, 21 Nov 2024 22:21:47 GMT, Phil Race wrote: > Clean up permission checks in java.awt.Desktop class > Also remove no longer needed exports from java.base to java.desktop There is a mention of the `GetPropertyAction` left in the comment: https://github.com/openjdk/jdk/blob/c199f5326b8dd41f

Re: RFR: 8344799: Remove permissions checks from java.awt.Desktop

2024-11-21 Thread Alexander Zvegintsev
On Thu, 21 Nov 2024 22:21:47 GMT, Phil Race wrote: > Clean up permission checks in java.awt.Desktop class > Also remove no longer needed exports from java.base to java.desktop Marked as reviewed by azvegint (Reviewer). src/java.desktop/share/classes/java/awt/Desktop.java line 409: > 407:

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-25 Thread Alexander Zvegintsev
On Fri, 18 Oct 2024 19:03:30 GMT, Sean Mullan wrote: >> This is the implementation of JEP 486: Permanently Disable the Security >> Manager. See [JEP 486](https://openjdk.org/jeps/486) for more details. The >> [CSR](https://bugs.openjdk.org/browse/JDK-8338412) describes in detail the >> main ch

Re: RFR: 8331605: jdk/test/lib/TestMutuallyExclusivePlatformPredicates.java test failure

2024-05-02 Thread Alexander Zvegintsev
On Thu, 2 May 2024 21:19:14 GMT, Phil Race wrote: > I see you added test.lib.Platform.isOnWayland() in this PR > https://github.com/openjdk/jdk/pull/18995/files#diff-a1fe81399728999601f16ae9aaf5caae27f009c394377d2cd86c71f6d5e0e54c > > So, that helps me understand why it is a new problem, but I

Integrated: 8331605: jdk/test/lib/TestMutuallyExclusivePlatformPredicates.java test failure

2024-05-02 Thread Alexander Zvegintsev
On Thu, 2 May 2024 21:05:38 GMT, Alexander Zvegintsev wrote: > Trivial fix. This pull request has now been integrated. Changeset: 01125fa2 Author: Alexander Zvegintsev URL: https://git.openjdk.org/jdk/commit/01125fa21b733199d4fe670ecf38b82cd917e242 Stats: 2 lines in 1 f

RFR: 8331605: jdk/test/lib/TestMutuallyExclusivePlatformPredicates.java test failure

2024-05-02 Thread Alexander Zvegintsev
Trivial fix. - Commit messages: - initial Changes: https://git.openjdk.org/jdk/pull/19071/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19071&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8331605 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: htt

Re: RFR: 8326062: ProblemList jcstress tests that are failing due to JDK-8325984

2024-02-16 Thread Alexander Zvegintsev
On Fri, 16 Feb 2024 15:54:27 GMT, Daniel D. Daugherty wrote: > A trivial fix to ProblemList jcstress tests that are failing due to > JDK-8325984. Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/17890#pullrequestreview-1885479796

Re: RFR: 8318038: ProblemList runtime/CompressedOops/CompressedClassPointers.java on two platforms

2023-10-12 Thread Alexander Zvegintsev
On Thu, 12 Oct 2023 19:26:10 GMT, Daniel D. Daugherty wrote: > Trivial ProblemListing for some tests: > > [JDK-8318038](https://bugs.openjdk.org/browse/JDK-8318038) ProblemList > runtime/CompressedOops/CompressedClassPointers.java on two platforms > [JDK-8318040](https://bugs.openjdk.org/brows

Re: RFR: 8314753: Remove support for @beaninfo, @ToDo, @since.unbundled, and @Note [v2]

2023-08-28 Thread Alexander Zvegintsev
On Tue, 22 Aug 2023 14:55:18 GMT, Pavel Rappo wrote: >> Please review this trivial PR. > > Pavel Rappo has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains two commits: > > - Merge branch 'master' into 8314753 > - Initial commit Mark

Re: Integrated: 8309236: ProblemList java/util/concurrent/locks/Lock/OOMEInAQS.java with ZGC and Generational ZGC again

2023-05-31 Thread Alexander Zvegintsev
On Wed, 31 May 2023 20:34:07 GMT, Daniel D. Daugherty wrote: > A trivial fix to ProblemList java/util/concurrent/locks/Lock/OOMEInAQS.java > with ZGC and Generational ZGC again Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/14253#pullrequ

Re: [jdk20] Integrated: 8298987: ProblemList jdk/internal/vm/Continuation/Fuzz.java#default with ZGC on X64

2022-12-17 Thread Alexander Zvegintsev
On Sat, 17 Dec 2022 16:00:32 GMT, Daniel D. Daugherty wrote: > A batch of trivial fixes to ProblemList tests: > [JDK-8298987](https://bugs.openjdk.org/browse/JDK-8298987) ProblemList > jdk/internal/vm/Continuation/Fuzz.java#default with ZGC on X64 > [JDK-8298989](https://bugs.openjdk.org/browse