Re: RFR: 8315097: Rename createJavaProcessBuilder

2023-08-28 Thread David Holmes
On Mon, 28 Aug 2023 15:54:08 GMT, Leo Korinth wrote: > Rename createJavaProcessBuilder so that it is not used by mistake instead of > createTestJvm. > > I have used the following sed script: `find -name "*.java" | xargs -n 1 sed > -i -e > "s/createJavaProcessBuilder(/createJavaProcessBuilderI

Re: RFR: 8314236: Overflow in Collections.rotate [v8]

2023-08-28 Thread Stuart Marks
On Fri, 25 Aug 2023 13:20:54 GMT, Nikita Sakharin wrote: >> `Collections.rotate` method contains a bug. This method throws >> IndexOutOfBoundsException on arrays larger than $2^{30}$ elements. The way >> to reproduce: >> >> final int size = (1 << 30) + 1; >> final List list = new ArrayList<>(s

Re: RFR: 8268829: Provide an optimized way to walk the stack with Class object only [v7]

2023-08-28 Thread Mandy Chung
> 8268829: Provide an optimized way to walk the stack with Class object only > > `StackWalker::walk` creates one `StackFrame` per frame and the current > implementation > allocates one `StackFrameInfo` and one `MemberName` objects per frame. Some > frameworks > like logging may only interest in

RFR: 8274122: java/io/File/createTempFile/SpecialTempFile.java fails in Windows 11

2023-08-28 Thread Brian Burkhalter
Windows 11 does not reserve as many names as prior versions of Windows so do not expect exceptions for COM7 and LPT1. - Commit messages: - 8274122: java/io/File/createTempFile/SpecialTempFile.java fails in Windows 11 Changes: https://git.openjdk.org/jdk/pull/15460/files Webrev: ht

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v30]

2023-08-28 Thread Erik Joelsson
On Mon, 28 Aug 2023 21:27:25 GMT, Srinivas Vamsi Parasa wrote: >> The goal is to develop faster sort routines for x86_64 CPUs by taking >> advantage of AVX512 instructions. This enhancement provides an order of >> magnitude speedup for Arrays.sort() using int, long, float and double arrays. >>

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v29]

2023-08-28 Thread Erik Joelsson
On Mon, 28 Aug 2023 22:42:50 GMT, Srinivas Vamsi Parasa wrote: >> make/modules/java.base/Lib.gmk line 239: >> >>> 237: >>> >>> 238: >>> 239: ifeq ($(call isTargetOs, linux)+$(call isTargetCpu, >>> x86_64)+$(INCL

Re: RFR: 8041488: Locale-Dependent List Patterns [v11]

2023-08-28 Thread Naoto Sato
> Introducing a new formatting class for locale-dependent list patterns. The > class is to provide the functionality from the Unicode Consortium's LDML > specification for [list > patterns](https://www.unicode.org/reports/tr35/tr35-general.html#ListPatterns). > For example, given a list of Stri

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v30]

2023-08-28 Thread Sandhya Viswanathan
On Mon, 28 Aug 2023 21:27:25 GMT, Srinivas Vamsi Parasa wrote: >> The goal is to develop faster sort routines for x86_64 CPUs by taking >> advantage of AVX512 instructions. This enhancement provides an order of >> magnitude speedup for Arrays.sort() using int, long, float and double arrays. >>

Re: CFV: New Core Libraries Group Member: Michael McMahon

2023-08-28 Thread Brent Christian
Vote: Yes On 8/25/23 8:24 AM, Roger Riggs wrote: I hereby nominate Michael McMahon to Membership in the Core Libraries Group

Re: CFV: New Core Libraries Group Member: Lance Andersen

2023-08-28 Thread Brent Christian
Vote: Yes On 8/25/23 8:23 AM, Roger Riggs wrote: I hereby nominate Lance Andersen to Membership in the Core Libraries Group

Re: CFV: New Core Libraries Group Member: Daniel Fuchs

2023-08-28 Thread Brent Christian
Vote: Yes On 8/25/23 8:23 AM, Roger Riggs wrote: I hereby nominate Daniel Fuchs to Membership in the Core Libraries Group

Re: RFR: JDK-8314611: Provide more explicative error message parsing Currencies [v3]

2023-08-28 Thread Justin Lu
> Please review this PR which updates some exceptions in j.util.Currency to > have an explicit error message (as opposed to nothing). > > The exceptions are thrown when the ISO 4217/3166 currency/country codes are > in an invalid form, or do not exist. Justin Lu has updated the pull request inc

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v29]

2023-08-28 Thread Srinivas Vamsi Parasa
On Fri, 25 Aug 2023 13:20:09 GMT, Erik Joelsson wrote: >> Srinivas Vamsi Parasa has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unnecessary import in Arrays.java > > make/modules/java.base/Lib.gmk line 239: > >> 237: >>

Re: RFR: JDK-8314611: Provide more explicative error message parsing Currencies [v2]

2023-08-28 Thread Naoto Sato
On Mon, 28 Aug 2023 21:36:34 GMT, Justin Lu wrote: >> Please review this PR which updates some exceptions in j.util.Currency to >> have an explicit error message (as opposed to nothing). >> >> The exceptions are thrown when the ISO 4217/3166 currency/country codes are >> in an invalid form, or

Re: RFR: JDK-8314611: Provide more explicative error message parsing Currencies [v2]

2023-08-28 Thread Justin Lu
> Please review this PR which updates some exceptions in j.util.Currency to > have an explicit error message (as opposed to nothing). > > The exceptions are thrown when the ISO 4217/3166 currency/country codes are > in an invalid form, or do not exist. Justin Lu has updated the pull request inc

RFR: 5066247: Refine the spec of equals() and hashCode() for j.text.Format classes

2023-08-28 Thread Justin Lu
Please review this PR which refines the spec of `equals()` and `hashCode()` in `java.text.Format` related classes. The current spec for most of these methods is either "_Overrides _" or are incomplete/wrong (i.e. see `ChoiceFormat`). This fix adjusts the spec to provide a consistent definition

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v26]

2023-08-28 Thread Srinivas Vamsi Parasa
On Fri, 25 Aug 2023 01:52:32 GMT, Sandhya Viswanathan wrote: >> pivotIndices array is being passed as a parameter to the partition intrinsic >> as it is updated in-place with the new pivot indices after partitioning. The >> Unsafe.ARRAY_INT_BASE_OFFSET is being used in libary_call.cpp to get t

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v29]

2023-08-28 Thread Srinivas Vamsi Parasa
On Fri, 25 Aug 2023 22:04:45 GMT, Vladimir Kozlov wrote: >> Srinivas Vamsi Parasa has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unnecessary import in Arrays.java > > src/hotspot/cpu/x86/stubGenerator_x86_64.cpp line 4143: > >>

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v30]

2023-08-28 Thread Srinivas Vamsi Parasa
> The goal is to develop faster sort routines for x86_64 CPUs by taking > advantage of AVX512 instructions. This enhancement provides an order of > magnitude speedup for Arrays.sort() using int, long, float and double arrays. > > This PR shows upto ~7x improvement for 32-bit datatypes (int, floa

RFR: JDK-8314611: Provide more explicative error message parsing Currencies

2023-08-28 Thread Justin Lu
Please review this PR which updates some exceptions in j.util.Currency to have an explicit error message (as opposed to nothing). The exceptions occur in the two static `getInstance()` methods. They are thrown generally when the ISO 4217/3166 currency/country codes are in an invalid form, or do

Re: RFR: 8314491: Linux: jexec launched via PATH fails to find java [v6]

2023-08-28 Thread Vladimir Petko
On Fri, 25 Aug 2023 07:04:50 GMT, Vladimir Petko wrote: >> 8314491: Linux: jexec launched via PATH fails to find java > > Vladimir Petko 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 me

Re: RFR: 8041488: Locale-Dependent List Patterns [v10]

2023-08-28 Thread Naoto Sato
On Mon, 28 Aug 2023 20:05:41 GMT, Naoto Sato wrote: >> Introducing a new formatting class for locale-dependent list patterns. The >> class is to provide the functionality from the Unicode Consortium's LDML >> specification for [list >> patterns](https://www.unicode.org/reports/tr35/tr35-genera

Re: RFR: 8041488: Locale-Dependent List Patterns [v10]

2023-08-28 Thread Naoto Sato
> Introducing a new formatting class for locale-dependent list patterns. The > class is to provide the functionality from the Unicode Consortium's LDML > specification for [list > patterns](https://www.unicode.org/reports/tr35/tr35-general.html#ListPatterns). > For example, given a list of Stri

Re: RFR: 8041488: Locale-Dependent List Patterns [v9]

2023-08-28 Thread Naoto Sato
> Introducing a new formatting class for locale-dependent list patterns. The > class is to provide the functionality from the Unicode Consortium's LDML > specification for [list > patterns](https://www.unicode.org/reports/tr35/tr35-general.html#ListPatterns). > For example, given a list of Stri

Re: RFR: 8297777: jdk.jlink StringSharingPlugin uses com.sun.tools.classfile library

2023-08-28 Thread Adam Sotona
On Wed, 23 Aug 2023 23:50:29 GMT, Qing Xiao wrote: > Migrate jdk.jlink StringSharingPlugin to use new classfile library It looks good, thanks. - PR Comment: https://git.openjdk.org/jdk/pull/15408#issuecomment-1695382042

RFR: 8297777: jdk.jlink StringSharingPlugin uses com.sun.tools.classfile library

2023-08-28 Thread Qing Xiao
Migrate jdk.jlink StringSharingPlugin to use new classfile library - Commit messages: - Migrate jdk.jlink StringSharingPlugin to use new classfile library Changes: https://git.openjdk.org/jdk/pull/15408/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15408&range=00 Issue:

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v5]

2023-08-28 Thread Daniel Fuchs
On Mon, 28 Aug 2023 18:49:40 GMT, Daniel Fuchs wrote: >> Sean Coffey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix up test cases > > test/jdk/java/lang/System/LoggerFinder/RecursiveLoading/PlatformRecursiveLoadingTest.java > line

Re: RFR: 8315097: Rename createJavaProcessBuilder

2023-08-28 Thread Leonid Mesnik
On Mon, 28 Aug 2023 15:54:08 GMT, Leo Korinth wrote: > Rename createJavaProcessBuilder so that it is not used by mistake instead of > createTestJvm. > > I have used the following sed script: `find -name "*.java" | xargs -n 1 sed > -i -e > "s/createJavaProcessBuilder(/createJavaProcessBuilderI

Re: RFR: 8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (int, long, float and double arrays) [v22]

2023-08-28 Thread Jatin Bhateja
On Fri, 25 Aug 2023 18:59:47 GMT, Srinivas Vamsi Parasa wrote: >>> Improvements are nice but it would not pay off if you have big regressions. >>> I can accept 0.9x but 0.4x - 0.8x regressions should be investigated and >>> implementation adjusted to avoid them. >> >> Hello Vladimir (@vnkozlo

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v5]

2023-08-28 Thread Daniel Fuchs
On Mon, 28 Aug 2023 18:45:03 GMT, Sean Coffey wrote: >> Recursive initialization calls possible during loading of LoggerFinder >> service. >> >> This fix detects the recursive call and returns a temporary LoggerFinder >> that is backed by a lazy logger. Automated test case developed to simul

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v4]

2023-08-28 Thread Daniel Fuchs
On Mon, 28 Aug 2023 17:29:16 GMT, Sean Coffey wrote: >> Recursive initialization calls possible during loading of LoggerFinder >> service. >> >> This fix detects the recursive call and returns a temporary LoggerFinder >> that is backed by a lazy logger. Automated test case developed to simul

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v4]

2023-08-28 Thread Daniel Fuchs
On Mon, 28 Aug 2023 17:42:34 GMT, Daniel Fuchs wrote: >> Sean Coffey has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Tidy up SignedLoggerFinderTest.java >> - Code contribution from Daniel included. New tests. Fix up extra service >>

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v5]

2023-08-28 Thread Sean Coffey
> Recursive initialization calls possible during loading of LoggerFinder > service. > > This fix detects the recursive call and returns a temporary LoggerFinder that > is backed by a lazy logger. Automated test case developed to simulate loading > of an external LoggerFinder service while als

Re: RFR: 8315117: Update Zlib Data Compression Library to Version 1.3

2023-08-28 Thread Alan Bateman
On Mon, 28 Aug 2023 17:15:36 GMT, Lance Andersen wrote: > It was a clean copy(which I have been able to do as of zlib 1.2.13). Sorry I > should have mentioned this in the PR request That's okay, I just wanted to check to be sure that the ChangeLog_java (renamed adler32.c -> zadler32.c, crc32c

Re: RFR: 8268829: Provide an optimized way to walk the stack with Class object only [v6]

2023-08-28 Thread Mandy Chung
On Fri, 25 Aug 2023 22:22:43 GMT, Mandy Chung wrote: >> 8268829: Provide an optimized way to walk the stack with Class object only >> >> `StackWalker::walk` creates one `StackFrame` per frame and the current >> implementation >> allocates one `StackFrameInfo` and one `MemberName` objects per fr

Re: RFR: 8268829: Provide an optimized way to walk the stack with Class object only [v6]

2023-08-28 Thread Mandy Chung
On Mon, 28 Aug 2023 14:47:20 GMT, Volker Simonis wrote: > thanks for doing this experiment. I've looked at your proposal, but I can't > see how it can help to fix > [JDK-8311500](https://bugs.openjdk.org/browse/JDK-8311500). I think you may be confused. This PR can be viewed as follows:

Re: CFV: New Core Libraries Group Member: Michael McMahon

2023-08-28 Thread Naoto Sato
Vote: yes Naoto On 8/25/23 8:24 AM, Roger Riggs wrote: I hereby nominate Michael McMahon to Membership in the Core Libraries Group Michael has been contributing to the OpenJDK Core Libraries since 2008, originally at Sun Microsystems, now Oracle. He has deep networking experience and has mad

Re: CFV: New Core Libraries Group Member: Daniel Fuchs

2023-08-28 Thread Naoto Sato
Vote: yes Naoto On 8/25/23 8:23 AM, Roger Riggs wrote: I hereby nominate Daniel Fuchs to Membership in the Core Libraries Group Daniel has been contributing to the OpenJDK Core Libraries at Oracle since 2012. He is leading the networking team and has made nearly 400 contributions to OpenJDK

Re: CFV: New Core Libraries Group Member: Lance Andersen

2023-08-28 Thread Naoto Sato
Vote: yes Naoto On 8/25/23 8:23 AM, Roger Riggs wrote: I hereby nominate Lance Andersen to Membership in the Core Libraries Group Lance has been contributing to the OpenJDK at Oracle since 2007. He has been the JDBC spec lead since 2005 and an OpenJDK committer since day 1. He has extensive

Re: RFR: 8314263: Signed jars triggering Logger finder recursion and StackOverflowError [v4]

2023-08-28 Thread Sean Coffey
> Recursive initialization calls possible during loading of LoggerFinder > service. > > This fix detects the recursive call and returns a temporary LoggerFinder that > is backed by a lazy logger. Automated test case developed to simulate loading > of an external LoggerFinder service while als

Re: RFR: 8315117: Update Zlib Data Compression Library to Version 1.3

2023-08-28 Thread Lance Andersen
On Mon, 28 Aug 2023 17:10:04 GMT, Alan Bateman wrote: > Are there any changes from the upstream zlib 1.3 code that should be called > out or is zlib 1.3 copied in without any changes? It was a clean copy(which I have been able to do as of zlib 1.2.13). Sorry I should have mentioned this in th

Re: RFR: 8315117: Update Zlib Data Compression Library to Version 1.3

2023-08-28 Thread Alan Bateman
On Mon, 28 Aug 2023 16:31:06 GMT, Lance Andersen wrote: > Hi all, > > Please review this PR which updates zlib from 1.2.13 to 1.3 in openJDK > > The [Zlib Data Compression Library](https://github.com/madler/zlib ) has > released Zlib 1.3 on August 18, 2023. > > There are a [small number of >

Withdrawn: 8314940: Use of NIO in JDKs Metrics implementation causes issues in GraalVM

2023-08-28 Thread Severin Gehwolf
On Thu, 24 Aug 2023 13:16:16 GMT, Severin Gehwolf wrote: > Please review this rather trivial fix to not use `nio` in `CgroupUtil`, part > of the > JDK's Metrics API. The primary motivating factor is that it allows one to use > the > JDK's version of `Metrics` in GraalVM. See the bug for details

Re: RFR: 8314940: Use of NIO in JDKs Metrics implementation causes issues in GraalVM

2023-08-28 Thread Severin Gehwolf
On Mon, 28 Aug 2023 15:29:32 GMT, Alan Bateman wrote: > > @AlanBateman Is there anything else you need me to do? If so, please let me > > know. Thanks! > > I don't think the JDK is the right place to workaround this issue. Also, we > really need to get back re-implementing FileInputStream and

RFR: 8315117: Update Zlib Data Compression Library to Version 1.3

2023-08-28 Thread Lance Andersen
Hi all, Please review this PR which updates zlib from 1.2.13 to 1.3 in openJDK The [Zlib Data Compression Library](https://github.com/madler/zlib ) has released Zlib 1.3 on August 18, 2023. There are a [small number of updates](https://github.com/madler/zlib/compare/v1.2.13...master ) between

RFR: 8315097: Rename createJavaProcessBuilder

2023-08-28 Thread Leo Korinth
Rename createJavaProcessBuilder so that it is not used by mistake instead of createTestJvm. I have used the following sed script: `find -name "*.java" | xargs -n 1 sed -i -e "s/createJavaProcessBuilder(/createJavaProcessBuilderIgnoreTestJavaOpts(/g"` Then I have manually modified ProcessTools.j

Re: RFR: 8314940: Use of NIO in JDKs Metrics implementation causes issues in GraalVM

2023-08-28 Thread Alan Bateman
On Fri, 25 Aug 2023 10:04:28 GMT, Alan Bateman wrote: >>> Something fishy here, is this working around a bug in GraaVM native image >>> or why does this change need to be in JDK? >> >> I've now realized that the bug had an incorrect statement in the >> description. The cycle happens due to the

Re: RFR: 8268829: Provide an optimized way to walk the stack with Class object only [v6]

2023-08-28 Thread Daniel Fuchs
On Fri, 25 Aug 2023 22:22:43 GMT, Mandy Chung wrote: >> 8268829: Provide an optimized way to walk the stack with Class object only >> >> `StackWalker::walk` creates one `StackFrame` per frame and the current >> implementation >> allocates one `StackFrameInfo` and one `MemberName` objects per fr

Re: RFR: 8314940: Use of NIO in JDKs Metrics implementation causes issues in GraalVM

2023-08-28 Thread Thomas Stuefe
On Thu, 24 Aug 2023 13:16:16 GMT, Severin Gehwolf wrote: > Please review this rather trivial fix to not use `nio` in `CgroupUtil`, part > of the > JDK's Metrics API. The primary motivating factor is that it allows one to use > the > JDK's version of `Metrics` in GraalVM. See the bug for details

Re: RFR: 8268829: Provide an optimized way to walk the stack with Class object only [v6]

2023-08-28 Thread Volker Simonis
On Fri, 25 Aug 2023 22:22:43 GMT, Mandy Chung wrote: >> 8268829: Provide an optimized way to walk the stack with Class object only >> >> `StackWalker::walk` creates one `StackFrame` per frame and the current >> implementation >> allocates one `StackFrameInfo` and one `MemberName` objects per fr

Re: RFR: 8314940: Use of NIO in JDKs Metrics implementation causes issues in GraalVM

2023-08-28 Thread Severin Gehwolf
On Fri, 25 Aug 2023 10:04:28 GMT, Alan Bateman wrote: >>> Something fishy here, is this working around a bug in GraaVM native image >>> or why does this change need to be in JDK? >> >> I've now realized that the bug had an incorrect statement in the >> description. The cycle happens due to the

Re: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread Oleksii Kucheruk
Yes, it's 2 subclasses hierarchy. https://github.com/iselo/jdk-optional/tree/master/lib/src I'm open to learning about my pitfalls. # JMH version: 1.36 # VM version: JDK 20.0.2, Java HotSpot(TM) 64-Bit Server VM, 20.0.2+9-78 # VM invoker: /Library/Java/JavaVirtualMachines/jdk-20.jdk/Contents/Hom

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: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread -
Can you share the benchmark code? Maybe C2 profiled class hierarchy, determined your optional only has 2 subclasses and can thus generate optimized machine code. Or maybe your benchmark is wrong that the result is not consumed by blackhole so the whole benchmark is moot. We need the code to make a

Re: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread Oleksii Kucheruk
Now it is clear. Sorry for the inconvenience. Thank you for your time. I appreciate. On Mon, 28 Aug 2023 at 2:03 PM Quân Anh Mai wrote: > Tbh I have a hard time understand what you are trying to convey. Please > write grammatically correct sentences with clear structure. > > I notice you attach

Re: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread Quân Anh Mai
Tbh I have a hard time understand what you are trying to convey. Please write grammatically correct sentences with clear structure. I notice you attach benchmark results but I don't see what was benchmarked to produce those results. Another point is that Value classes has not been delivered yet,

Re: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread Oleksii Kucheruk
Thank you Quân. All you saying guys make sense. Yes there is a difference calling methods by pointer and involving vtable. But one thing didn't came out of my head: If virtual dispatch dispatch is 10-time more expensive and polymorphism is a performance killer so how combinations of "return optio

Re: yield return based on Continuations

2023-08-28 Thread Daniel Schmid
Thank you for your interest, I'll also include loom-dev with this mail. Yours, Daniel Am 28.08.2023 um 11:12 schrieb Alan Bateman: This looks fun! It's probably best to bring this to loom-dev. In its archives you'll find several discussions about generators as several people have been inter

Re: RFR: 8314491: Linux: jexec launched via PATH fails to find java [v6]

2023-08-28 Thread Alan Bateman
On Fri, 25 Aug 2023 07:04:50 GMT, Vladimir Petko wrote: >> 8314491: Linux: jexec launched via PATH fails to find java > > Vladimir Petko 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 me

Re: yield return based on Coroutines

2023-08-28 Thread Alan Bateman
This looks fun! It's probably best to bring this to loom-dev. In its archives you'll find several discussions about generators as several people have been interested in that topic. Even when thread confined, the main concern has been that exotic control flow yields leads to surprising behavio

yield return based on Coroutines

2023-08-28 Thread Daniel Schmid
Hi, After seeing the JVM Language Summit talk on Continuations (https://www.youtube.com/watch?v=6nRS6UiN7X0), I thought about it being possible to implement something like "yield return" in languages like C# (or "yield" in Python) based on Continuations. Kotlin has implemented a similar featur

Re: RFR: 8294969: Convert jdk.jdeps javap to use the Classfile API [v11]

2023-08-28 Thread Adam Sotona
On Thu, 20 Jul 2023 09:11:20 GMT, Adam Sotona wrote: >> javap uses proprietary com.sun.tools.classfile library to parse class files. >> >> This patch converts javap to use Classfile API. >> >> Please review. >> >> Thanks, >> Adam > > Adam Sotona has updated the pull request with a new target b

Re: RFE: Refactor java.util.Optional and add NonNull checks

2023-08-28 Thread Quân Anh Mai
Hi, Polymorphism is a performance killer. Normally this does not matter much, but for thin wrappers such as Optional, this is one of the most important factor. Behaviour polymorphism requires virtual dispatch and prevents inlining. This is really detrimental, as simple operations such as a getter

Re: RFR: JDK-8314272: Improve java.util.prefs.BackingStoreException: Couldn't get file lock.

2023-08-28 Thread Matthias Baesken
On Tue, 22 Aug 2023 07:48:46 GMT, Matthias Baesken wrote: > For the tests, I'm surprised the TEST.properties in this directory doesn't > have exclusiveAccess.dirs=. as there has been historical issues with > interference between tests in this area. Hi Alan , when adding the test group sun/to

Re: RFR: JDK-8313764: Offer JVM HS functionality to shared lib load operations done by the JDK codebase [v2]

2023-08-28 Thread Matthias Baesken
On Mon, 28 Aug 2023 02:26:30 GMT, David Holmes wrote: > Sorry but looking at the hotspot part of this I do not like the code in > jvm.cpp at all - it is far too messy. I expected to see a simple interface to > os::dlopen which then handles all the platform specific issues. > > I'm also somewha

Re: CFV: New Core Libraries Group Member: Michael McMahon

2023-08-28 Thread Per-Ake Minborg
Vote: yes -Per From: core-libs-dev on behalf of Roger Riggs Sent: Friday, August 25, 2023 5:24 PM To: core-libs-dev Subject: CFV: New Core Libraries Group Member: Michael McMahon I hereby nominate Michael McMahon to Membership in the Core Libraries Group Mich

Re: CFV: New Core Libraries Group Member: Daniel Fuchs

2023-08-28 Thread Per-Ake Minborg
Vote: yes * Per From: core-libs-dev on behalf of Roger Riggs Sent: Friday, August 25, 2023 5:23 PM To: core-libs-dev Subject: CFV: New Core Libraries Group Member: Daniel Fuchs I hereby nominate Daniel Fuchs to Membership in the Core Libraries Group Dan

Re: CFV: New Core Libraries Group Member: Lance Andersen

2023-08-28 Thread Per-Ake Minborg
Vote: yes -Per From: core-libs-dev on behalf of Roger Riggs Sent: Friday, August 25, 2023 5:23 PM To: core-libs-dev Subject: CFV: New Core Libraries Group Member: Lance Andersen I hereby nominate Lance Andersen to Membership in the Core Libraries Group Lance