On Fri, 18 Apr 2025 21:26:23 GMT, Joe Wang wrote:
>> Fix a NPE on calling DocumentBuilderFactory::getAttribute, refer to the bug
>> report.
>>
>> Also in this patch: consolidates get and set properties to use the same Util
>> methods to reduce potential errors when code changes.
>>
>> Test:
>
On Fri, 18 Apr 2025 20:10:32 GMT, Brian Burkhalter wrote:
>> Implement the requested methods and add a test thereof.
>
> Brian Burkhalter has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8354724: Remove unused import
test/jdk/java/io/Buffer
On Fri, 28 Mar 2025 01:08:42 GMT, Brent Christian wrote:
>> I propose some cleanups to `FinalizerHistogramTest.java` to hopefully clear
>> up the intermittent failures:
>>
>> * run with `othervm`: this test blocks the (global) finalizer thread, and
>> also requires the (global) finalizer threa
We have helpful NPE messages now - they are more user-friendly.
And shorter methods are more likely to be inlined.
-
Commit messages:
- [PATCH] Avoid explicit Objects.requireNonNull where we know implicit NPE
will happen
- [PATCH] Avoid explicit Objects.requireNonNull where we know
On Thu, 6 Mar 2025 12:38:01 GMT, Jaikiran Pai wrote:
>> We have helpful NPE messages now - they are more user-friendly.
>> And shorter methods are more likely to be inlined.
>
> src/java.base/share/classes/java/lang/String.java line 3649:
>
>> 3647: Iterable elements) {
>> 3648:
On Thu, 20 Feb 2025 09:30:02 GMT, Andrey Turbanov wrote:
> We have helpful NPE messages now - they are more user-friendly.
> And shorter methods are more likely to be inlined.
While having prerequirements checks often is good, I think not having it in
String.join can be a good th
On Wed, 5 Mar 2025 21:13:44 GMT, Alexey Semenyuk wrote:
> jpackage app laucnher will expand environment variables in .cfg files.
>
> Previously jpackage app launcher only replaced `$APPDIR`, `$BINDIR`, and
> `$ROOTDIR` tokens with the corresponding path values. With this patch, any
> environme
On Thu, 6 Mar 2025 10:46:11 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enabl
On Fri, 14 Feb 2025 02:23:40 GMT, SendaoYan wrote:
> Hi all,
>
> The newly added JMH tests
> 'org.openjdk.bench.jdk.incubator.vector.VectorMultiplyOptBenchmark' fails
> "java.lang.NoClassDefFoundError: jdk/incubator/vector/Float16" by below test
> command:
>
>
> make test MICRO="FORK=1;WARM
On Mon, 3 Feb 2025 18:58:57 GMT, Justin Lu wrote:
>> Please review this PR which improves the performance of cut-over date
>> checking when the user supplies a properties override via the
>> `java.util.currency.data` sys prop. Replacing the `SimpleDateFormat` with a
>> _java.time_ alternative
On Fri, 31 Jan 2025 19:46:37 GMT, Justin Lu wrote:
>> Please review this PR which improves the performance of cut-over date
>> checking when the user supplies a properties override via the
>> `java.util.currency.data` sys prop. Replacing the `SimpleDateFormat` with a
>> _java.time_ alternative
On Tue, 16 Jan 2024 10:19:44 GMT, Andrey Turbanov wrote:
> There are 3 methods in `java.util.TimeZone` which are `public static` and
> marked as `synchronized`:
> 1. getTimeZone(String)
> 2. getAvailableIDs(int)
> 3. getAvailableIDs()
>
> This means it is a bottle neck for
On Thu, 23 Jan 2025 22:48:17 GMT, Chen Liang wrote:
>> Right, it can be removed. Actually, I think `getAvailableIDs()` and
>> `getAvailableIDs(int)` also do not need to be synchronized. The internal IDs
>> are eagerly initialized in the static block of `ZoneInfoFile.java` and are
>> effectivel
ementation and concluded that `synchronized` is
> unnecessary.
Andrey Turbanov 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 remote-tracking branch 'refs/remotes/origin/master' into
JDK-8291027-TimeZone-sy
On Wed, 18 Dec 2024 21:07:14 GMT, Roger Riggs wrote:
>> SM Cleanup of common test library test/lib/...:
>>
>> Remove unnecessary catches of SecurityException
>> Remove AccessController and doPrivileged from SimpleSSLContext and
>> ProcessTools.
>
> Roger Riggs has updated the pull request incre
On Thu, 9 Jan 2025 08:40:37 GMT, Adam Sotona wrote:
>> There are no more consumers of ASM library except for hotspot tests.
>> This patch moves ASM library from java.base module to the hotspot test
>> libraries location and fixes the tests.
>>
>> Please review.
>>
>> Thanks,
>> Adam
>
> Adam S
On Thu, 2 Jan 2025 18:11:53 GMT, Naoto Sato wrote:
>> The change made in
>> [JDK-8288723](https://bugs.openjdk.org/browse/JDK-8288723) seems innocuous,
>> but it caused this performance regression. Partially reverting the change
>> (ones that involve `computeIfAbsent()`) to the original. Provi
On Thu, 2 Jan 2025 18:11:53 GMT, Naoto Sato wrote:
>> The change made in
>> [JDK-8288723](https://bugs.openjdk.org/browse/JDK-8288723) seems innocuous,
>> but it caused this performance regression. Partially reverting the change
>> (ones that involve `computeIfAbsent()`) to the original. Provi
On Thu, 26 Dec 2024 16:39:58 GMT, Brett Okken wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fixed compile error
>
> src/java.base/share/classes/java/time/format/DateTimeTextProvider.java line
> 316:
>
>> 314:
On Tue, 17 Dec 2024 11:09:09 GMT, Jatin Bhateja wrote:
>> Hi All,
>>
>> This patch adds C2 compiler support for various Float16 operations added by
>> [PR#22128](https://github.com/openjdk/jdk/pull/22128)
>>
>> Following is the summary of changes included with this patch:-
>>
>> 1. Detection
On Tue, 5 Nov 2024 08:28:10 GMT, Andrey Turbanov wrote:
> The field `com.sun.jndi.ldap.EventSupport#notifiers` is always accessed under
> `lock`. It means extra synchronization from `Hashtable` is not needed.
> Subtle difference in Hashtable vs Hashmap behavior is that Hashtabl
On Thu, 21 Nov 2024 10:24:28 GMT, Daniel Fuchs wrote:
>> This PR remove usage of SecurityManager, doPrivileges, etc... from
>> `java.logging` and `java.base/jdk.internal.logger`
>>
>> Only notable hack - Logger.checkPermission() no longer checks permissions,
>> but has been renamed into `ensur
On Tue, 19 Nov 2024 22:31:28 GMT, Alexey Semenyuk wrote:
> Stop using internal AppImageFile, ApplicationLayout, and PackageFile classes
> in the tests. Replace them with stubs.
>
> Stop referencing IOUtils class from XmlUtils and PathUtils classes.
test/jdk/tools/jpackage/helpers/jdk/jpackage/
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian wrote:
>> Remove usages of SecurityManager, doPrivildged, and AccessController from
>> the java.prefs module.
>
> Brent Christian has updated the pull request incrementally with one
> additional commit since the last revision:
>
> indentation
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian wrote:
>> Remove usages of SecurityManager, doPrivildged, and AccessController from
>> the java.prefs module.
>
> Brent Christian has updated the pull request incrementally with one
> additional commit since the last revision:
>
> indentation
On Tue, 19 Nov 2024 20:43:25 GMT, Sean Mullan wrote:
> Now that JEP 486 has been integrated, `java.security.jgss` and
> `jdk.security.jgss` module dependencies on `System.getSecurityManager` and
> `AccessController.doPrivileged*` can be removed.
>
> There is an undocumented property named "sun
The field `com.sun.jndi.ldap.EventSupport#notifiers` is always accessed under
`lock`. It means extra synchronization from `Hashtable` is not needed.
Subtle difference in Hashtable vs Hashmap behavior is that Hashtable doesn't
allow `null` keys and `null` values. I've checked all usages of it - on
On Tue, 5 Nov 2024 12:49:04 GMT, Andrey Turbanov wrote:
> Content of Hashtable `com.sun.jndi.ldap.LdapClient#defaultBinaryAttrs` is
> fully initialized in `` block.
> It means we can replace legacy synchronized `Hashtable` with immutable set.
This pull request has now been i
On Tue, 5 Nov 2024 12:49:04 GMT, Andrey Turbanov wrote:
> Content of Hashtable `com.sun.jndi.ldap.LdapClient#defaultBinaryAttrs` is
> fully initialized in `` block.
> It means we can replace legacy synchronized `Hashtable` with immutable set.
I run `com/sun/jndi` tests on my win x64.
I wounder, why java.lang.Math.toIntExact accepts only long parameter?
I think it will quite useful to have it with float/double parameters too.
Andrey Turbanov
On Tue, 12 Nov 2024 21:59:43 GMT, Roger Riggs wrote:
>> After [JDK-8338411](https://bugs.openjdk.org/browse/JDK-8338411),
>> Serialization implementation dependencies on SecurityManager, doPrivildged,
>> and AccessController are removed.
>> Some refactoring to cleanup the remaining code is expe
On Tue, 12 Nov 2024 16:30:12 GMT, Shaojin Wen wrote:
>> In the JDK code, there are some places that may cause Unsafe offset
>> overflow. The probability of occurrence is low, but if it occurs, it will
>> cause JVM crash.
>
> Shaojin Wen has updated the pull request incrementally with one additi
On Tue, 12 Nov 2024 18:05:46 GMT, Roger Riggs wrote:
>> Refactor removing the dependencies on SecurityManager, doPrivileged, and
>> AccessController.
>
> Roger Riggs has updated the pull request with a new target base due to a
> merge or a rebase. The pull request now contains 228 commits:
>
>
Content of Hashtable `com.sun.jndi.ldap.LdapClient#defaultBinaryAttrs` is fully
initialized in `` block.
It means we can replace legacy synchronized `Hashtable` with immutable set.
-
Commit messages:
- [PATCH] Unnecessary Hashtable usage in LdapClient.defaultBinaryAttrs
Changes: ht
ps, merely querying the map with get
is a structural modification."
As I understand, it means that LinkedHashMap.get can't be guarded
_just_ by ReadLock.
Is my assumption correct?
Andrey Turbanov
On Mon, 21 Oct 2024 14:25:06 GMT, Raffaello Giulietti
wrote:
> This helps in reducing memory consumption, especially for long inputs.
src/java.base/share/classes/jdk/internal/math/FloatingDecimal.java line 1875:
> 1873: } // look for and process decimal floating-point string
> 187
On Thu, 17 Oct 2024 14:28:30 GMT, Patricio Chilano Mateo
wrote:
> This is the implementation of JEP 491: Synchronize Virtual Threads without
> Pinning. See [JEP 491](https://bugs.openjdk.org/browse/JDK-8337395) for
> further details.
>
> In order to make the code review easier the changes hav
On Thu, 31 Oct 2024 07:15:16 GMT, Taizo Kurashige wrote:
> To resolve java/io/File/GetXSpace.java failure, I fix libGetXSpace.c to use
> Cygwin’s `df` to get the size for comparison if the test target drive is a
> CD-ROM drive.
>
> As described in JDK-8343342, GetDiskSpaceInformationW can't g
On Thu, 24 Oct 2024 03:01:54 GMT, Ioi Lam wrote:
>> This is an implementation of [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://openjdk.org/jeps/483).
>>
>>
>> Note: this is a combined PR of the following individual PRs
>> - https://github.com/openjdk/jdk/pull/20516
>> - https
On Thu, 24 Oct 2024 13:52:57 GMT, Claes Redestad wrote:
> Many OpenJDK micros use `@Fork(jvmArgs/-Append/-Prepend)` to add JVM
> reasonable or necessary flags, but when deploying and running micros we often
> want to add or replace flags to tune to the machine, test different GCs, etc.
> The i
On Mon, 23 Sep 2024 09:13:58 GMT, Andrey Turbanov wrote:
> 8342083: Make a few fields in FileSystemPreferences final
This pull request has now been integrated.
Changeset: 0e3fc93d
Author: Andrey Turbanov
URL:
https://git.openjdk.org/jdk/com
On Thu, 17 Oct 2024 14:28:30 GMT, Patricio Chilano Mateo
wrote:
> This is the implementation of JEP 491: Synchronize Virtual Threads without
> Pinning. See [JEP 491](https://bugs.openjdk.org/browse/JDK-8337395) for
> further details.
>
> In order to make the code review easier the changes hav
On Sun, 20 Oct 2024 18:59:36 GMT, Joe Darcy wrote:
>> Port of Float16 from java.lang in the lworld+fp16 branch to
>> jdk.incubabor.vector.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Remove comments for intrinsics per rev
8342083: Make a few fields in FileSystemPreferences final
-
Commit messages:
- [PATCH] Make a few fields in FileSystemPreferences final
Changes: https://git.openjdk.org/jdk/pull/21131/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=21131&range=00
Issue: https://bugs.openj
On Thu, 10 Oct 2024 20:36:26 GMT, fabioromano1 wrote:
>> An optimized algorithm for `BigDecimal.stripTrailingZeros()` that uses
>> repeated squares trick.
>
> fabioromano1 has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Refining mathematic
On Mon, 23 Sep 2024 18:45:49 GMT, Ioi Lam wrote:
> This is the 7th and final PR for [JEP 483: Ahead-of-Time Class Loading &
> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>
> This PR implements the AOT-linking of invokedynamic callsites:
> - We only link lambda expressions (`LambdaMet
On Fri, 20 Sep 2024 02:53:00 GMT, Alexey Semenyuk wrote:
>> Move `resource.wxl-file-name` property from `WinResources.properties` files
>> to the new `WinResourcesNoL10N.properties` files. The later are supposed to
>> be excluded from automatic translations.
>
> Alexey Semenyuk has updated the
On Wed, 11 Sep 2024 00:29:30 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal of this PR is to implement an x86_64 intrinsic for
>> java.lang.Math.tanh() using libm
>>
>> Benchmark (ops/ms) | Stock JDK | Tanh intrinsic | Speedup
>> -- | -- | -- | --
>> MathBench.tanhDouble | 70900 | 95618 | 1.35x
On Thu, 29 Aug 2024 17:06:19 GMT, Andrey Turbanov wrote:
>> My suspicion is that Condition::await() throws before having successfully
>> reacquired the lock, and this exception is swallowed because Lock::unlock()
>> then throws when invoke with an IllegalMonitorStateExcept
On Thu, 15 Aug 2024 18:22:54 GMT, Viktor Klang wrote:
> My suspicion is that Condition::await() throws before having successfully
> reacquired the lock, and this exception is swallowed because Lock::unlock()
> then throws when invoke with an IllegalMonitorStateException as the current
> thread
On Wed, 28 Aug 2024 01:49:21 GMT, Chen Liang wrote:
> Make `DirectCodeBuilder` write instructions actually directly without
> allocating extra objects. This speed up a lot of simple Class-File building
> cases that never go through intermediate transforms.
src/java.base/share/classes/java/lang
On Tue, 20 Aug 2024 18:52:23 GMT, Viktor Klang wrote:
> Unfortunately there is no good, deterministic reproducer which can be used as
> a regression test at this point in time.
There is now typo in interface name. `ScheuledThreadPoolExecutor` ->
`ScheduledThreadPoolExecutor`
-
PR
On Wed, 14 Aug 2024 04:59:23 GMT, Jatin Bhateja wrote:
>> Hi All,
>>
>> As per the discussion on panama-dev mailing list[1], patch adds the support
>> following new vector operators.
>>
>>
>> . SUADD : Saturating unsigned addition.
>> . SADD: Saturating signed addition.
>>
On Thu, 8 Aug 2024 06:57:28 GMT, Jatin Bhateja wrote:
> Hi All,
>
> As per the discussion on panama-dev mailing list[1], patch adds the support
> for following new two vector permutation APIs.
>
>
> Declaration:-
> Vector.selectFrom(Vector v1, Vector v2)
>
>
> Semantics:-
> Using in
On Thu, 25 Jul 2024 08:59:17 GMT, Andrey Turbanov wrote:
> 3 fields in java.text.MergeCollation could be made 'static':
> 1. BITARRAYMASK
> 2. BYTEPOWER
> 3. BYTEMASK
This pull request has now been integrated.
Changeset: 60c9b5cd
Author:Andrey Turbanov
URL:
h
On Fri, 26 Jul 2024 13:09:19 GMT, Andrey Turbanov wrote:
> There is implicit null check in line before.
>
> https://github.com/openjdk/jdk/blob/431d4f7e18369466eedd00926a5162a1461d0b25/src/java.base/share/classes/java/io/ObjectOutputStream.java#L1267-L1277
>
> 'cl'
On Fri, 26 Jul 2024 13:09:19 GMT, Andrey Turbanov wrote:
> There is implicit null check in line before.
>
> https://github.com/openjdk/jdk/blob/431d4f7e18369466eedd00926a5162a1461d0b25/src/java.base/share/classes/java/io/ObjectOutputStream.java#L1267-L1277
>
> 'cl'
On Tue, 6 Aug 2024 19:46:02 GMT, Roger Riggs wrote:
>> A few methods in `java.time.chrono.ChronoLocalDateImpl` are unused and could
>> be removed:
>> 1. plusWeeks(long)
>> 2. minusYears(long)
>> 3. minusMonths(long)
>> 4. minusWeeks(long)
>> 5. minusDays(long)
>>
>> Tested `test/jdk/java/time`
There is implicit null check in line before.
https://github.com/openjdk/jdk/blob/431d4f7e18369466eedd00926a5162a1461d0b25/src/java.base/share/classes/java/io/ObjectOutputStream.java#L1267-L1277
'cl' can't be null after that.
-
Commit messages:
- [PATCH] Remove redundant null check
3 fields in java.text.MergeCollation could be made 'static':
1. BITARRAYMASK
2. BYTEPOWER
3. BYTEMASK
-
Commit messages:
- [PATCH] Make a few fields in MergeCollation static
Changes: https://git.openjdk.org/jdk/pull/20323/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=2032
A few methods in `java.time.chrono.ChronoLocalDateImpl` are unused and could be
removed:
1. plusWeeks(long)
2. minusYears(long)
3. minusMonths(long)
4. minusWeeks(long)
5. minusDays(long)
Tested `test/jdk/java/time` on Linux x64 release
-
Commit messages:
- [PATCH] Remove unused me
On Mon, 29 Jul 2024 02:22:11 GMT, Chen Liang wrote:
>> As discussed in offline meeting, the max stack and locals information are
>> part of the code attribute and not meaningful for buffered code elements.
>> Computation would be costly and these see no real usage during
>> transformations. Th
On Wed, 24 Jul 2024 13:00:47 GMT, Chen Liang wrote:
>> `TypeAnnotation` is not an annotation, as it should not be used in places
>> like `AnnotationValue.ofAnnotation`. Thus it's remodeled to contain an
>> annotation at a given location instead of to be an annotation.
>>
>> Depends on #20205.
On Tue, 23 Jul 2024 20:54:08 GMT, Justin Lu wrote:
>> As discussed in
>> https://github.com/openjdk/jdk/pull/19513#discussion_r1684957942, there is
>> code within java.text.NumberFormat (and subclasses) that can use the pattern
>> match switch to improve readability.
>>
>> As this is simply c
On Mon, 15 Jul 2024 00:50:30 GMT, Axel Boldt-Christmas
wrote:
>> When inflating a monitor the `ObjectMonitor*` is written directly over the
>> `markWord` and any overwritten data is displaced into a displaced
>> `markWord`. This is problematic for concurrent GCs which needs extra care or
>> l
On Thu, 11 Jul 2024 17:30:21 GMT, Alan Bateman wrote:
> Bringover some of the changes accumulated in the loom repo to the main line,
> most of these changes are test updates and have been baking in the loom repo
> for several months. The motive is partly to reduce the large set of changes
> th
On Tue, 16 Jul 2024 16:54:09 GMT, Naoto Sato wrote:
>> Removing a redundant private method, which has the same implementation with
>> the public sibling and obsolete method description.
>
> Naoto Sato has updated the pull request incrementally with one additional
> commit since the last revisio
On Wed, 10 Jul 2024 22:08:47 GMT, Justin Lu wrote:
>Please review this PR, which is a backport of commit
>[86b0cf25](https://github.com/openjdk/jdk/commit/86b0cf259fb3cbe3a1973151148e5d36c6a99d91)
> from the [openjdk/jdk](https://git.openjdk.org/jdk) repository.
This message is kind of confusin
On Fri, 28 Jun 2024 12:39:32 GMT, Claes Redestad wrote:
>> This PR attains a speed-up on some microbenchmarks by simplifying how we
>> build up the MH combinator tree shape
>> (only use prependers with prefix, always add a suffix to the newArray
>> combinator), then simplifying/inlining some of
On Tue, 18 Jun 2024 02:00:41 GMT, SendaoYan wrote:
>> Hi all,
>> Testcase
>> `test/jdk/java/util/ResourceBundle/Control/MissingResourceCauseTestRun.java`
>> run fails with root user privileged. I think it's necessary to skip this
>> testcase when user is root.
>> Why run the jtreg test by root
On Wed, 12 Jun 2024 14:08:43 GMT, Ferenc Rakoczi wrote:
>> This PR removes some unnecessary conversions between byte arrays and long
>> arrays during SHA3 digest computations.
>
> Ferenc Rakoczi has updated the pull request incrementally with one additional
> commit since the last revision:
>
On Wed, 22 May 2024 05:16:42 GMT, Jaikiran Pai wrote:
> Can I please get a review of this test-only change for addressing
> https://bugs.openjdk.org/browse/JDK-8332490?
>
> The jmh test opens a `InflaterInputStream`, reads the stream contents, but
> then doesn't close the stream. This can lead
On Tue, 21 May 2024 21:16:52 GMT, Pavel Rappo wrote:
>> src/java.base/share/classes/jdk/internal/io/JdkConsoleImpl.java line 78:
>>
>>> 76: String line = null;
>>> 77: synchronized (writeLock) {
>>> 78: synchronized(readLock) {
>>
>> Suggestion:
>>
>> sy
On Tue, 21 May 2024 15:44:18 GMT, Pavel Rappo wrote:
>> Please review this PR which introduces the `java.io.IO` top-level class and
>> three methods to `java.io.Console` for [Implicitly Declared Classes and
>> Instance Main Methods (Third Preview)].
>>
>> This PR has been obtained as `git merg
On Thu, 21 Mar 2024 17:13:46 GMT, Bill Huang wrote:
>> This task addresses an essential aspect of our testing infrastructure: the
>> proper handling and cleanup of temporary files and socket files created
>> during test execution. The motivation behind these changes is to prevent the
>> accumu
On Tue, 19 Mar 2024 00:07:37 GMT, Justin Lu wrote:
>> Please review this PR and associated
>> [CSR](https://bugs.openjdk.org/browse/JDK-8327703) which introduces strict
>> parsing for NumberFormat.
>>
>> The concrete subclasses that will utilize this leniency value are
>> `DecimalFormat` and
l request incrementally with one additional
> commit since the last revision:
>
> Delete extra space
>
> Co-authored-by: Andrey Turbanov
test/jdk/java/foreign/CallGeneratorHelper.java line 195:
> 193: }
> 194:
> 195: private static void generateStructDecl
On Tue, 12 Mar 2024 14:06:11 GMT, Aggelos Biboudis
wrote:
> In cases where the compiler needs to unbox a `long`, `float`, `double` and
> then run the exactness check, we were getting a crash. While the selector
> value is always boxed, the type (which controls the execution flow) was not,
> b
On Mon, 11 Mar 2024 17:17:21 GMT, Joe Darcy wrote:
>> The atan2 and hypot cases added would fail for a particular test library
>> that has errors in the millions of ulps for those inputs, rather than the
>> small number of ulps specified for the error.
>
> Joe Darcy has updated the pull request
On Wed, 6 Mar 2024 20:49:55 GMT, Joe Darcy wrote:
>> The atan2 and hypot cases added would fail for a particular test library
>> that has errors in the millions of ulps for those inputs, rather than the
>> small number of ulps specified for the error.
>
> Joe Darcy has updated the pull request
On Tue, 16 Jan 2024 10:19:44 GMT, Andrey Turbanov wrote:
> There are 3 methods in `java.util.TimeZone` which are `public static` and
> marked as `synchronized`:
> 1. getTimeZone(String)
> 2. getAvailableIDs(int)
> 3. getAvailableIDs()
>
> This means it is a bottle neck for
On Fri, 23 Feb 2024 21:24:10 GMT, Naoto Sato wrote:
> This PR intends to remove the legacy `COMPAT` locale data from the JDK. The
> `COMPAT` locale data was introduced for applications' migratory purposes
> transitioning to `CLDR`. It is becoming a technical debt and now is the time
> to remov
On Thu, 15 Feb 2024 19:44:42 GMT, Justin Lu wrote:
> Please review this PR which handles an edge case pattern bug with
> ChoiceFormat.
>
>
> var d = new ChoiceFormat("0#foo|1#bar|baz|") // creates cFmt equivalent to
> "0.0#foo|1.0#bar|1.0#"
> d.format(1) // unexpectedly returns ""
>
>
> Not
On Tue, 13 Feb 2024 18:03:14 GMT, Dan Lutker wrote:
>> Ran the test on AmazonLinux 2 which has multiple binaries from coreutils
>> package and no coreutils executable as well as AmazonLinux 2023 that uses
>> `--enable-single-binary`
>
> Dan Lutker has updated the pull request incrementally with
On Thu, 8 Feb 2024 13:18:41 GMT, Adam Sotona wrote:
> ClassFile API provides two sets of instructions implementations (bound and
> unbound).
> Unbound implementation of `IncrementInstruction::constant` returns invalid
> value.
> This bug discovered a hole in the ClassFile API test coverage.
>
On Fri, 2 Feb 2024 22:43:28 GMT, Justin Lu wrote:
>> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319344)
>> which adds MessageFormat pattern support for the following subformats:
>> ListFormat, CompactNumberFormat, and DateTimeFormatter. This change is
>> intended to p
On Wed, 17 Jan 2024 18:19:18 GMT, Naoto Sato wrote:
>> src/java.base/share/classes/java/util/TimeZone.java line 629:
>>
>>> 627: */
>>> 628: public static String[] getAvailableIDs(int rawOffset) {
>>> 629: return ZoneInfo.getAvailableIDs(rawOffset);
>>
>> BTW can we call `ZoneI
On Tue, 16 Jan 2024 10:19:44 GMT, Andrey Turbanov wrote:
> 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily
src/java.base/share/classes/java/util/TimeZone.java line 629:
> 627: */
> 628: public static String[] getAvailableIDs(int rawOffset) {
&g
8291027: Some of TimeZone methods marked 'synchronized' unnecessarily
-
Commit messages:
- 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily
Changes: https://git.openjdk.org/jdk/pull/17441/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17441&range=00
I
On Wed, 10 Jan 2024 13:39:52 GMT, Eirik Bjørsnøs wrote:
>> ZipInputStream.readEnd currently assumes a Zip64 data descriptor if the
>> number of compressed or uncompressed bytes read from the inflater is larger
>> than the Zip64 magic value.
>>
>> While the ZIP format mandates that the data de
On Thu, 11 Jan 2024 23:06:32 GMT, Scott Gibbons wrote:
>> Re-write the IndexOf code without the use of the pcmpestri instruction, only
>> using AVX2 instructions. This change accelerates String.IndexOf on average
>> 1.3x for AVX2. The benchmark numbers:
>>
>>
>> Benchmark
On Thu, 14 Dec 2023 04:00:58 GMT, Vicente Romero wrote:
>> Reflection is not retrieving generic type information for mandated
>> parameters. This is a known issue which has been explicitly stated in the
>> API of some reflection methods. Fix for
>> [JDK-8292275](https://bugs.openjdk.org/browse
On Tue, 2 Jan 2024 12:21:21 GMT, Eirik Bjørsnøs wrote:
>> Please consider this PR which makes `DeflaterOutputStream.close()` always
>> close its wrapped output stream exactly once.
>>
>> Currently, closing of the wrapped output stream happens outside the finally
>> block where `finish()` is ca
On Tue, 9 Jan 2024 16:48:56 GMT, Jatin Bhateja wrote:
>> Hi,
>>
>> Patch optimizes non-subword vector compress and expand APIs for x86 AVX2
>> only targets.
>> Upcoming E-core Xeons (Sierra Forest) and Hybrid CPUs only support AVX2
>> instruction set.
>> These are very frequently used APIs in
On Tue, 9 Jan 2024 13:30:58 GMT, Adam Sotona wrote:
>> ClassFile API performance related improvements have been separated from
>> #17121 into this PR.
>>
>> These improvements are important to minimize performance regression of
>> 8294961: Convert java.base/java.lang.reflect.ProxyGenerator to u
On Fri, 12 Jan 2024 15:06:01 GMT, Chris Hegarty wrote:
>> Update LinkedTransferQueue add and put methods to not call overridable offer.
>
> Chris Hegarty has updated the pull request incrementally with one additional
> commit since the last revision:
>
> timed offer
src/java.base/share/class
On Tue, 12 Dec 2023 15:45:47 GMT, Per Minborg wrote:
>> This PR proposes to change the specification for some methods that take
>> `long` offsets so that they will throw an `IllegalArgumentException` rather
>> than an `IndexOutOfBoundsException` for negative values.
>>
>> The PR also proposes
On Tue, 5 Dec 2023 14:55:02 GMT, Viktor Klang wrote:
>> Splits GatherersTest out to distinct files per built-in gatherer
>>
>> Fixes a test ordering issue for mapConcurrent which placed GatherersTest on
>> the ProblemList.
>>
>> Adding increased maxOutputSize for Gatherer-related tests to impr
On Thu, 30 Nov 2023 23:49:24 GMT, Joe Darcy wrote:
>> Time to start making preparations for JDK 23.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Update symbol files to JDK 22 b26.
test/langtools/tools/javac/versions/Versio
On Fri, 17 Nov 2023 18:46:57 GMT, Andrey Turbanov wrote:
>> Viktor Klang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Addressing trailing whitespace complaint
>
> src/java.base/share/classes/java/util/s
1 - 100 of 342 matches
Mail list logo
