On Mon, 18 Nov 2024 20:37:27 GMT, Chen Liang wrote:
>> Justin Lu has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> address Chen's review; make other similar changes
>
> src/java.base/share/classes/java/util/Locale.java line 94:
>
>> 92:
On 18/11/2024 21:17, Chen Liang wrote:
Hi Eirik,
I think it might have been a legacy oversight; note that even
java.util.Arrays was just recently made final.
> If Runtime was made final it seems we could remove the CheckCSMs test.
I disagree; CheckCSMs test checks for missing CSM annotation
On Fri, 15 Nov 2024 18:32:58 GMT, Roger Riggs wrote:
> This cleanup of the use of SecurityManager is in the jdk.unsupported module.
> Removed the permission check to call getReflectionFactory().
Marked as reviewed by alanb (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/2
On Mon, 18 Nov 2024 21:06:11 GMT, Sean Mullan wrote:
> Is this worthy of a release note?
There are hundreds of methods changed to no longer do permission checks,
remaining permission checks do nothing anyway. I would think the big release
note for JEP 486 will provide an overall summary of th
On Tue, 19 Nov 2024 06:13:04 GMT, David Holmes wrote:
>> src/java.base/share/man/java.md line 3079:
>>
>>> 3077: For example `*.java` stays `*.java` and is not expanded to
>>> `Foo.java`,
>>> 3078: `Bar.java`, etc. like on some command line shell.
>>> 3079:
>>
>> This expansion now se
On Mon, 28 Oct 2024 13:53:58 GMT, Jorn Vernee wrote:
> There is a subtle race in `UpcallLinker::on_exit` between copying of the old
> frame anchor back into place, and the GC. Since this copy is not atomic, it
> may briefly appear as if a thread has no last Java frame, while still in the
> `_t
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
src/java.base/share/man/java.md line 3079:
> 3077: For example `*.java` stays `*.java` and is not expanded to
> `Foo.java`,
On Tue, 19 Nov 2024 06:12:21 GMT, David Holmes wrote:
>> Please review this improvement to the manual page covering the usage of
>> wildcards in `java`'s argfiles.
>
> src/java.base/share/man/java.md line 3079:
>
>> 3077: For example `*.java` stays `*.java` and is not expanded to
>> `Foo.j
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
No, that was an oversight. Good catch!
I will remove that sentence (second last bullet point) in favor of the
augmented one (thi
> Can I please get a review of this change, which simplifies the interaction
> between the `java` launcher's native code with the
> `sun.launcher.LauncherHelper`?
>
> As noted in https://bugs.openjdk.org/browse/JDK-8341184, this proposed change
> reduces the back and forth between the launcher
On Sun, 17 Nov 2024 19:01:24 GMT, Eirik Bjørsnøs wrote:
> Please review this PR which cleans up SecurityManager-related code in
> `java.sql` and `java.sql.rowset` modules post JEP-486
>
> There are quite a few changes to review, but all relatively straightforward:
>
> `DriverManager`
> * Remov
On Mon, 28 Oct 2024 13:53:58 GMT, Jorn Vernee wrote:
> There is a subtle race in `UpcallLinker::on_exit` between copying of the old
> frame anchor back into place, and the GC. Since this copy is not atomic, it
> may briefly appear as if a thread has no last Java frame, while still in the
> `_t
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
Hello Christian, one part of the confusion noted in that linked issue
https://bugs.openjdk.org/browse/JDK-8293040 is:
> Wildcar
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
Marked as reviewed by wkora...@github.com (no known OpenJDK username).
-
PR Review: https://git.openjdk.org/jdk/pull
On Mon, 18 Nov 2024 09:46:50 GMT, Aleksey Shipilev wrote:
>> See the bug for more discussion and reproducer. This PR replaces the ad-hoc
>> linked list with segmented list of arrays. Arrays are easy targets for GC.
>> There are possible improvements here, most glaring is parallelism that is
>>
On Fri, 15 Nov 2024 19:30:06 GMT, Aleksey Shipilev wrote:
>> See the bug for more discussion and reproducer. This PR replaces the ad-hoc
>> linked list with segmented list of arrays. Arrays are easy targets for GC.
>> There are possible improvements here, most glaring is parallelism that is
>>
> This PR permanently disable remote code downloading in JNDI/LDAP and JNDI/RMI
> JDK providers, and contains the following changes:
> - The following two properties are removed:
> - `com.sun.jndi.ldap.object.trustURLCodebase`
> - `com.sun.jndi.rmi.object.trustURLCodebase`
> - JNDIs object
On Mon, 18 Nov 2024 20:34:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Mon, 18 Nov 2024 15:50:49 GMT, Jan Lahoda wrote:
>> Currently, running `java` without any parameters will lead to an output that
>> is a full `--help`, which is over 100 lines (on my computer at least), and
>> it feels overwhelming. And many people might actually want to run
>> JShell/REPL,
On Mon, 18 Nov 2024 21:56:12 GMT, Brian Burkhalter wrote:
>> Expunge the use of the `SecurityManager`, `AccessController`, and the like
>> from the `java.io` package.
>
> Brian Burkhalter has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 834
On Mon, 18 Nov 2024 18:31:09 GMT, Alexey Semenyuk wrote:
> Move child classes of "jdk.jpackage.test.Functional" class to
> "jdk.jpackage.internal.util.function" package.
> Split "jdk.jpackage.internal.IOUtils" class in "PathUtils", "FileUtils",
> "XmlUtils" classes in "jdk.jpackage.internal.uti
On Mon, 18 Nov 2024 21:56:12 GMT, Brian Burkhalter wrote:
>> Expunge the use of the `SecurityManager`, `AccessController`, and the like
>> from the `java.io` package.
>
> Brian Burkhalter has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 834
On Sun, 17 Nov 2024 14:52:06 GMT, Eirik Bjørsnøs wrote:
> Please review this PR which removes exceptional control flow in
> `Boolean::getBoolean`, `Integer::getInteger` and `Long::getLong`.
>
> These methods are catching `IllegalArgumentException` and
> `NullPointerException`, thrown by `Syste
On Mon, 18 Nov 2024 13:41:49 GMT, Alexey Semenyuk wrote:
> Support using jpackage l10n keys instead of string values when examining
> jpackage error output.
Looks good.
-
Marked as reviewed by almatvee (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22207#pullrequestrevie
> We would like to propose an implementation of the HKDF algorithms for
> SunPKCS11, aligned with the KDF API proposed for JDK 24 (see [JEP 478: Key
> Derivation Function API
> (Preview)](https://bugs.openjdk.org/browse/JDK-8189808)).
>
> This implementation will be under the _Preview_ umbrella
On Mon, 18 Nov 2024 21:57:04 GMT, Stuart Marks wrote:
>> First cut at removal of Security Manager stuff from RMI.
>>
>> This covers just about every SM-related case in RMI, except for a bit of
>> package checking in MarshalInputStream. This will be handled separately.
>> It's covered by [JDK-8
On Mon, 18 Nov 2024 15:45:22 GMT, Alexey Semenyuk wrote:
> jpackage man page update
Looks good.
-
Marked as reviewed by almatvee (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22209#pullrequestreview-2443917267
On Mon, 18 Nov 2024 20:34:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Mon, 18 Nov 2024 13:21:25 GMT, Alexey Semenyuk wrote:
> Simple typo fix.
>
> Old view:
> https://github.com/openjdk/jdk/blob/dfddbcaab886b9baa731cd748bb7f547e1903b64/src/jdk.jpackage/share/man/jpackage.md
>
> New view:
> https://github.com/alexeysemenyukoracle/jdk/blob/JDK-8336087/src/jdk.
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
A straightforward example. Don't know which area this belongs to exactly,
though.
-
Marked as reviewed by liach (R
On Mon, 18 Nov 2024 21:07:59 GMT, Brian Burkhalter wrote:
>> Expunge the use of the `SecurityManager`, `AccessController`, and the like
>> from the `java.io` package.
>
> Brian Burkhalter has updated the pull request incrementally with two
> additional commits since the last revision:
>
> - 8
> First cut at removal of Security Manager stuff from RMI.
>
> This covers just about every SM-related case in RMI, except for a bit of
> package checking in MarshalInputStream. This will be handled separately. It's
> covered by [JDK-8344329](https://bugs.openjdk.org/browse/JDK-8344329).
>
> Fu
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
Brian Burkhalter has updated the pull request incrementally with one additional
commit since the last revision:
836: Revert spurious inclusion of SCTP changes
-
Changes
On Sat, 16 Nov 2024 00:03:23 GMT, Justin Lu wrote:
>> Please review this PR which removes the lone `AccessController.doPrivileged`
>> call from the _java.scripting_ module after JEP486 integration.
>
> Justin Lu has updated the pull request incrementally with one additional
> commit since the l
On Mon, 18 Nov 2024 15:22:59 GMT, Christian Stein wrote:
> Please review this improvement to the manual page covering the usage of
> wildcards in `java`'s argfiles.
This pull request has now been integrated.
Changeset: 5cb0d438
Author:Christian Stein
URL:
https://git.openjdk.org/jd
On Mon, 18 Nov 2024 18:49:57 GMT, Roger Riggs wrote:
>> First cut at removal of Security Manager stuff from RMI.
>>
>> This covers just about every SM-related case in RMI, except for a bit of
>> package checking in MarshalInputStream. This will be handled separately.
>> It's covered by [JDK-83
On Mon, 18 Nov 2024 17:39:44 GMT, Aleksei Efimov wrote:
>> First cut at removal of Security Manager stuff from RMI.
>>
>> This covers just about every SM-related case in RMI, except for a bit of
>> package checking in MarshalInputStream. This will be handled separately.
>> It's covered by [JDK
On Mon, 18 Nov 2024 18:05:12 GMT, Aleksei Efimov wrote:
>> src/java.rmi/share/classes/sun/rmi/server/MarshalInputStream.java line 69:
>>
>>> 67:
>>> 68: /** table to hold sun classes to which access is explicitly
>>> permitted */
>>> 69: protected static Map> permittedSunClasses
>>
>>
On Fri, 15 Nov 2024 23:23:39 GMT, Justin Lu wrote:
> Please review this PR which removes the lone `AccessController.doPrivileged`
> call from the _java.scripting_ module after JEP486 integration.
This pull request has now been integrated.
Changeset: 8d43e0d6
Author:Justin Lu
URL:
h
On Mon, 18 Nov 2024 21:20:02 GMT, Chen Liang wrote:
>> Eirik Bjørsnøs has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Remove ObjectStreamField#getType from UNSUPPORTED_VIRTUAL_METHODS, a
>> leftover from JDK-8344034
>
> test/jdk/jdk/int
On Mon, 18 Nov 2024 20:34:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Mon, Nov 18, 2024 at 10:17 PM Chen Liang wrote:
> Hi Eirik,
> I think it might have been a legacy oversight; note that even
> java.util.Arrays was just recently made final.
>
Thanks, filed https://bugs.openjdk.org/browse/JDK-8344460 to track this
enhancement.
> > If Runtime was made final i
On Mon, 18 Nov 2024 21:07:59 GMT, Brian Burkhalter wrote:
>> Expunge the use of the `SecurityManager`, `AccessController`, and the like
>> from the `java.io` package.
>
> Brian Burkhalter has updated the pull request incrementally with two
> additional commits since the last revision:
>
> - 8
Hi Eirik,
I think it might have been a legacy oversight; note that even java.util.Arrays
was just recently made final.
> If Runtime was made final it seems we could remove the CheckCSMs test.
I disagree; CheckCSMs test checks for missing CSM annotations when new caller
sensitive methods are add
> As a prerequisite for Hermetic Java, we need a statically linked `java`
> launcher. It should behave like the normal, dynamically linked `java`
> launcher, except that all JDK native libraries should be statically, not
> dynamically, linked.
>
> This patch is the first step towards this goal.
On Mon, 18 Nov 2024 07:41:31 GMT, Justin Lu wrote:
> Please review this PR and corresponding CSR which includes a wide range of
> specification improvements for java.util.Locale. See the CSR for further
> detail. Other changes/suggestions are welcomed to be included as part of this
> change. A
On Fri, 15 Nov 2024 18:32:58 GMT, Roger Riggs wrote:
> This cleanup of the use of SecurityManager is in the jdk.unsupported module.
> Removed the permission check to call getReflectionFactory().
Marked as reviewed by mullan (Reviewer).
Is this worthy of a release note?
-
PR Review
On Mon, 18 Nov 2024 15:55:56 GMT, Jan Lahoda wrote:
>> So, just to double-check: given the request is to make the helps consistent,
>> and dropping the extensions from the concise help is not what was meant,
>> I'll add the extensions both here and to the ordinary help. Correct?
>
> Done here:
On Mon, 18 Nov 2024 20:46:24 GMT, Roger Riggs wrote:
>> Brian Burkhalter has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - 836: Address review comments
>> - 836: Remove security manager dependency from module jdk.sctp
>
> src/ja
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
Brian Burkhalter has updated the pull request incrementally with two additional
commits since the last revision:
- 836: Address review comments
- 836: Remove security manager dep
On Mon, 18 Nov 2024 20:56:59 GMT, Naoto Sato wrote:
>> Brian Burkhalter has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - 836: Address review comments
>> - 836: Remove security manager dependency from module jdk.sctp
>
> src/jav
On Wed, 13 Nov 2024 17:05:25 GMT, Magnus Ihse Bursie wrote:
> Currently, the man pages are stored as troff (a text format) in the open
> repo, and a content-wise identical copy is stored as markdown (another text
> format) in the closed repo.
>
> Since markdown is preferred to troff in terms o
On Mon, 18 Nov 2024 09:42:20 GMT, Alan Bateman wrote:
>> Change default value of the command line option
>> --sun-misc-unsafe-memory-access from "allow" to "warn".
>
> Alan Bateman has updated the pull request with a new target base due to a
> merge or a rebase. The pull request now contains si
On Mon, 18 Nov 2024 07:41:31 GMT, Justin Lu wrote:
> Please review this PR and corresponding CSR which includes a wide range of
> specification improvements for java.util.Locale. See the CSR for further
> detail. Other changes/suggestions are welcomed to be included as part of this
> change. A
On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter wrote:
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
src/java.base/share/classes/java/io/Console.java line 660:
> 658: }
> 659:
> 660: @SuppressWarnings("removal")
This too
On Mon, 18 Nov 2024 20:35:49 GMT, Sean Mullan wrote:
> What about `sun.misc.Unsafe`? There are a couple of
> `AccessController.doPrivileged` calls.
I'll look at that in a separate PR.
-
PR Comment: https://git.openjdk.org/jdk/pull/22158#issuecomment-2484111717
On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter wrote:
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
`Console` changes look good to me, with a minor suggestion
src/java.base/share/classes/java/io/Console.java line 687:
> 685:
On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter wrote:
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
Looks good
src/java.base/share/classes/java/io/FilePermission.java line 184:
> 182: DefaultFileSystemProvider.theFileSys
On Mon, 18 Nov 2024 20:34:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
> Please review this PR which cleans up SecurityManager-related code in
> `java.sql` and `java.sql.rowset` modules post JEP-486
>
> There are quite a few changes to review, but all relatively straightforward:
>
> `DriverManager`
> * Remove `SecurityManager::checkPermission` calls in the `setLogW
On Fri, 15 Nov 2024 18:32:58 GMT, Roger Riggs wrote:
> This cleanup of the use of SecurityManager is in the jdk.unsupported module.
> Removed the permission check to call getReflectionFactory().
What about `sun.misc.Unsafe`? There are a couple of
`AccessController.doPrivileged` calls.
Hi,
Tried to find an answer to this question in JBS, but found nothing:
Since java.lang.Runtime has a single private constructor, it seems
effectively final. Why is it not declared final?
If Runtime was made final it seems we could remove the CheckCSMs test.
Are there compatibility concerns her
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter wrote:
> Expunge the use of the `SecurityManager`, `AccessController`, and the like
> from the `java.io` package.
Preliminary testing: the `jdk_io` tests pass on Linux and Windows.
-
PR Comment: https://git.openjdk.org/jdk/pull/22
On Mon, 18 Nov 2024 20:27:20 GMT, Roger Riggs wrote:
> Yes, please cleanup the leftover mention, in CheckCSMs. of
> ObjectStreamField.getType(). And I'll re-review.
Thanks, see 0278c59. This leaves `UNSUPPORTED_VIRTUAL_METHODS` as an empty set.
Should I remove this field and the code using thi
Expunge the use of the `SecurityManager`, `AccessController`, and the like from
the `java.io` package.
-
Commit messages:
- 8344077: Remove security manager dependency in java.io
Changes: https://git.openjdk.org/jdk/pull/22219/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&p
On Mon, 18 Nov 2024 19:47:25 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Mon, 18 Nov 2024 19:30:08 GMT, Roger Riggs wrote:
>> Eirik Bjørsnøs has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Revert caller-sensitive inlining of ReflectUtil.forName
>> - SerialJavaObject::getFields is no longer @CallerSensit
Improve documentation for constant pool entries. This include some models for
those entries, symbolic descriptors, and some general guidance in package
summary.
APIDiff: https://cr.openjdk.org/~liach/apidiff/cf-consts/
Javadoc: https://cr.openjdk.org/~liach/javadoc/cf-consts/
Please review the
> Please review this PR which cleans up SecurityManager-related code in
> `java.sql` and `java.sql.rowset` modules post JEP-486
>
> There are quite a few changes to review, but all relatively straightforward:
>
> `DriverManager`
> * Remove `SecurityManager::checkPermission` calls in the `setLogW
On Mon, 18 Nov 2024 08:37:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Mon, 18 Nov 2024 12:40:43 GMT, Per Minborg wrote:
> This PR proposes to use a static property already resolved instead of calling
> `System.getProperty()`.
The CSR for this change has been approved. I plan to integrate this tomorrow
pending a final tier2 run.
-
PR Comment: htt
On Mon, 18 Nov 2024 14:11:43 GMT, Raffaello Giulietti
wrote:
>> fabioromano1 has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Optimize sqrt branch for exact results
>
> src/java.base/share/classes/java/math/BigDecimal.java line 2248:
>
On Fri, 15 Nov 2024 17:03:50 GMT, Aleksei Efimov wrote:
> This PR permanently disable remote code downloading in JNDI/LDAP and JNDI/RMI
> JDK providers, and contains the following changes:
> - The following two properties are removed:
> - `com.sun.jndi.ldap.object.trustURLCodebase`
> - `
> Removed use of SecurityManager to check permissions.
> Relaxed defensive copying of byte arrays (due to lack of SM)
Roger Riggs has updated the pull request incrementally with one additional
commit since the last revision:
Review suggestions applied:
Remove SecurityExceptions from LiveStac
On Fri, 15 Nov 2024 19:26:15 GMT, Naoto Sato wrote:
> Removing a `AccessController.doPrivileged()` call from jdk.charsets module
> after JEP486 integration.
This pull request has now been integrated.
Changeset: c59adf68
Author:Naoto Sato
URL:
https://git.openjdk.org/jdk/commit/c59a
On Mon, 18 Nov 2024 08:37:26 GMT, Eirik Bjørsnøs wrote:
>> Please review this PR which cleans up SecurityManager-related code in
>> `java.sql` and `java.sql.rowset` modules post JEP-486
>>
>> There are quite a few changes to review, but all relatively straightforward:
>>
>> `DriverManager`
>>
On Fri, 15 Nov 2024 01:12:34 GMT, Stuart Marks wrote:
> First cut at removal of Security Manager stuff from RMI.
>
> This covers just about every SM-related case in RMI, except for a bit of
> package checking in MarshalInputStream. This will be handled separately. It's
> covered by [JDK-834432
On Mon, 18 Nov 2024 18:31:09 GMT, Alexey Semenyuk wrote:
> Move child classes of "jdk.jpackage.test.Functional" class to
> "jdk.jpackage.internal.util.function" package.
> Split "jdk.jpackage.internal.IOUtils" class in "PathUtils", "FileUtils",
> "XmlUtils" classes in "jdk.jpackage.internal.uti
Move child classes of "jdk.jpackage.test.Functional" class to
"jdk.jpackage.internal.util.function" package.
Split "jdk.jpackage.internal.IOUtils" class in "PathUtils", "FileUtils",
"XmlUtils" classes in "jdk.jpackage.internal.util" package.
Implmenetation didnt change except of `FileUtils.delet
On Sat, 26 Oct 2024 14:06:37 GMT, Naman Nigam wrote:
> As the documentation of getCeilingEntry currently reads, the inference is
> misleading as follows:
>
> if no such entry exists, returns the entry for the least key greater than the
> specified key;
> if no such entry exists (i.e., the grea
On Thu, 14 Nov 2024 03:23:29 GMT, Naman Nigam wrote:
>> As the documentation of getCeilingEntry currently reads, the inference is
>> misleading as follows:
>>
>> if no such entry exists, returns the entry for the least key greater than
>> the specified key;
>> if no such entry exists (i.e., th
We would like to propose an implementation of the HKDF algorithms for
SunPKCS11, aligned with the KDF API proposed for JDK 24 (see [JEP 478: Key
Derivation Function API
(Preview)](https://bugs.openjdk.org/browse/JDK-8189808)).
This implementation will be under the _Preview_ umbrella until the K
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote:
>> Removing a `AccessController.doPrivileged()` call from jdk.charsets module
>> after JEP486 integration.
>
> Naoto Sato has updated the pull request incrementally with one additional
> commit since the last revision:
>
> fixed import
Tha
On Thu, 14 Nov 2024 03:23:29 GMT, Naman Nigam wrote:
>> As the documentation of getCeilingEntry currently reads, the inference is
>> misleading as follows:
>>
>> if no such entry exists, returns the entry for the least key greater than
>> the specified key;
>> if no such entry exists (i.e., th
On Fri, 15 Nov 2024 01:12:34 GMT, Stuart Marks wrote:
> First cut at removal of Security Manager stuff from RMI.
>
> This covers just about every SM-related case in RMI, except for a bit of
> package checking in MarshalInputStream. This will be handled separately. It's
> covered by [JDK-834432
On Mon, 18 Nov 2024 17:55:31 GMT, Aleksei Efimov wrote:
>> First cut at removal of Security Manager stuff from RMI.
>>
>> This covers just about every SM-related case in RMI, except for a bit of
>> package checking in MarshalInputStream. This will be handled separately.
>> It's covered by [JDK
On Fri, 15 Nov 2024 01:12:34 GMT, Stuart Marks wrote:
> First cut at removal of Security Manager stuff from RMI.
>
> This covers just about every SM-related case in RMI, except for a bit of
> package checking in MarshalInputStream. This will be handled separately. It's
> covered by [JDK-834432
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
On Fri, 15 Nov 2024 01:12:34 GMT, Stuart Marks wrote:
> First cut at removal of Security Manager stuff from RMI.
>
> This covers just about every SM-related case in RMI, except for a bit of
> package checking in MarshalInputStream. This will be handled separately. It's
> covered by [JDK-834432
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
Can I get a review for this test only change to the Since Checker?
I drop the distinction between classes and interfaces when generating ids and
use a generic name "class" to describe both, as to not consider classes that
get converted to interfaces (and vice versa) as new API (Something that ma
On Mon, 18 Nov 2024 15:46:04 GMT, Viktor Klang wrote:
> This change now halves each side of the split regardless, the old code would
> end up in a situation where it wouldn't decrement since unsigned shift right
> twice would lead to decrementing estimated size by 0.
>
> It's worth noting the
First cut at removal of Security Manager stuff from RMI.
This covers just about every SM-related case in RMI, except for a bit of
package checking in MarshalInputStream. This will be handled separately. It's
covered by [JDK-8344329](https://bugs.openjdk.org/browse/JDK-8344329).
Further simplifi
On Mon, 18 Nov 2024 12:40:43 GMT, Per Minborg wrote:
> This PR proposes to use a static property already resolved instead of calling
> `System.getProperty()`.
This pull request has now been integrated.
Changeset: 20783295
Author:Per Minborg
URL:
https://git.openjdk.org/jdk/commit/2
> Please review this PR which cleans up SecurityManager-related code in
> `java.sql` and `java.sql.rowset` modules post JEP-486
>
> There are quite a few changes to review, but all relatively straightforward:
>
> `DriverManager`
> * Remove `SecurityManager::checkPermission` calls in the `setLogW
On Fri, 15 Nov 2024 18:32:58 GMT, Roger Riggs wrote:
> This cleanup of the use of SecurityManager is in the jdk.unsupported module.
> Removed the permission check to call getReflectionFactory().
CSR created, please review:
[JDK-8344423](https://bugs.openjdk.org/browse/JDK-8344423)
On Mon, 18 Nov 2024 15:50:49 GMT, Jan Lahoda wrote:
>> Currently, running `java` without any parameters will lead to an output that
>> is a full `--help`, which is over 100 lines (on my computer at least), and
>> it feels overwhelming. And many people might actually want to run
>> JShell/REPL,
1 - 100 of 151 matches
Mail list logo
