On Fri, 2 May 2025 18:19:35 GMT, Brian Burkhalter wrote:
>> This change proposes to modify `java.io.File.delete()` so that regular files
>> on Windows will not be deleted by default if their read-only attribute is
>> set. A boolean-valued system compatibility property
>> `jdk.io.File.deleteRea
You need to drain those streams in two dedicated threats - or in some cases use the redirect feature to pipe it to the stdout of the parent (good option for automation scripting)I think without a helper ProcessBuilder is hard to use for such cases -- https://bernd.eckenfels.netFrom: core-libs-dev
On Wed, 2 Apr 2025 18:39:57 GMT, Severin Gehwolf wrote:
> For JEP 493-enabled builds there are no JMODs. Certain files come from the
> installed JDK image when a user creates a custom run-time from it. This is
> problematic for example for files that often come from a different package
> (e.g.
On Wed, 2 Apr 2025 18:39:57 GMT, Severin Gehwolf wrote:
> For JEP 493-enabled builds there are no JMODs. Certain files come from the
> installed JDK image when a user creates a custom run-time from it. This is
> problematic for example for files that often come from a different package
> (e.g.
You can just switch-match on trusted types, then it even looks modern. :)Gruß,Bernd-- https://bernd.eckenfels.net Von: core-libs-dev im Auftrag von Chen Liang Gesendet: Montag, März 17, 2025 3:25 AMAn: core-libs-dev@openjdk.org ; nio-...@openjdk.org Betreff: Re: RFR: 8343110: Add getChars(int
On Tue, 11 Feb 2025 17:50:45 GMT, Jamil Nimeh wrote:
> This fix makes some minor changes to the internals of the
> `CertificateBuilder` and `SimpleOCSPServer` test classes. They would break
> when ML-DSA was selected as key and signing algorithms. Also RSASSA-PSS
> works better now with thes
Hello,In the past I startet out with Application Code to generate UUIDs, then moved to the JCL Version for randomUUID() and enjoyed the benefit of speed and no additional depenemcy. However random v4 always had the problem of ruining Index locality in databases. So now I would be in the Position to
On Thu, 10 Oct 2024 21:12:50 GMT, Markus KARG wrote:
> I have slightly adapted the copy loop to allow CPUs to use solely
> single-operand INC within the loop instead of dual-operand ADD.
If that makes a difference we might need to consider fixing the Compiler :)
-
PR Comment: http
On Sun, 6 Oct 2024 18:05:28 GMT, Bernd wrote:
>> Markus KARG has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> fixup! Reader.of(String)
>>
>> Dropping non-public JavaDocs
>
> src/java.base/
On Sun, 6 Oct 2024 17:44:53 GMT, Markus KARG wrote:
>> This Pull Requests proposes an implementation for
>> [JDK-8341566](https://bugs.openjdk.org/browse/JDK-8341566): Adding the new
>> method `public static Reader Reader.of(CharSequence)` will return an
>> anonymous, non-synchronized implemen
On Sun, 6 Oct 2024 17:44:53 GMT, Markus KARG wrote:
>> This Pull Requests proposes an implementation for
>> [JDK-8341566](https://bugs.openjdk.org/browse/JDK-8341566): Adding the new
>> method `public static Reader Reader.of(CharSequence)` will return an
>> anonymous, non-synchronized implemen
On Sun, 6 Oct 2024 17:31:42 GMT, Markus KARG wrote:
>> I recommend adding another test case against an ad-hoc `CharSequence`
>> implementation wrapping a `char[]` in a record, to ensure the generic paths
>> in `read(char[], int, int)` works as intended.
>
> Good idea. But instead, we could also
On Sun, 6 Oct 2024 17:44:53 GMT, Markus KARG wrote:
>> This Pull Requests proposes an implementation for
>> [JDK-8341566](https://bugs.openjdk.org/browse/JDK-8341566): Adding the new
>> method `public static Reader Reader.of(CharSequence)` will return an
>> anonymous, non-synchronized implemen
On Sun, 6 Oct 2024 14:54:22 GMT, Markus KARG wrote:
>> src/java.base/share/classes/java/io/Reader.java line 163:
>>
>>> 161: * {@code transferTo()} methods all throw {@code IOException}.
>>> 162: *
>>> 163: * The {@code markSupported()} method returns {@code true}.
>>
>> Suggest
char[] does not implement CharSequence)
You hinted at it, but just to make it explicit, user mode implementations
cant do the needed optimizations to avoid the char[] clone.
Gruss
Bernd
>
Why not use codepoints() instead, then you dont need to worry about surrogates and all methods which work on char should also exist for codepoints.(I know that’s not your point but what I am trying to say there is a reason for this API decision “char must die”. although chars() i
Hello,To me bytecode is a standardized interface and creating or modifying bytecode is legal and not inherently unsafe.If annotation processor authors want to use bytecode modifications they now get an additional tool for that but it doesn’t change the general usage.In the end us
On Sat, 14 Sep 2024 13:14:35 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this change which cleans up the `java`
>> launcher to remove checks/support for outdated options?
>>
>> As noted in https://bugs.openjdk.org/browse/JDK-8339918, these 6 options
>> have been outdated and unsu
On Sat, 14 Sep 2024 13:14:35 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this change which cleans up the `java`
>> launcher to remove checks/support for outdated options?
>>
>> As noted in https://bugs.openjdk.org/browse/JDK-8339918, these 6 options
>> have been outdated and unsu
On Sat, 14 Sep 2024 13:14:35 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this change which cleans up the `java`
>> launcher to remove checks/support for outdated options?
>>
>> As noted in https://bugs.openjdk.org/browse/JDK-8339918, these 6 options
>> have been outdated and unsu
has the myapp\app\A.cfg file in there, so I
am not sure what the Exception is - does it try to start it?
Gruss
Bernd
PS: can i change the native launcher to not search the config in app\ but in
its own directory and also have it relative to bin\ directory (on Windows
similar to Linux). It d
/share/classes/java/util/stream/ReferencePipeline.java#L696
The special things with steams is, that the Stream class only sets up the
evaluation pipeline.
Gruss
Bernd
Anil wrote on 21. July 2024 21:19 (GMT +02:00):
> As mentioned in my post, I know where the package is, but I cannot find
&g
Anil wrote on 21. July 2024 04:44 (GMT +02:00):
> However, I am unable to find the implementing class of java.util.Stream.
You find the package in Java.base module:
https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/java/util/stream
Gruß
Bernd
—
https://bernd.eckenfels.net
On Sat, 8 Jun 2024 06:40:39 GMT, Shaojin Wen wrote:
>> After PR #16245, C2 optimizes stores into primitive arrays by combining
>> values into larger stores. In the UUID.toString method,
>> ByteArrayLittleEndian can be removed, making the code more elegant and
>> faster.
>
> Shaojin Wen has u
Hello Archie,
Archie Cobbs wrote on 09.05.2024 20:09 (GMT +02:00):
FWIW when I run the test case in that bug on the latest JDK 23 (jdk-23+21-110-g0a4eeeaa3c6), it does not report any exception.
So this bug may have gotten fixed by recent changes and just needs to be closed... ?
Interesting ob
Hello,
I remember to have seen a few ZIP64 compatibility bug fixed in the past.
But this one here is still open:
* [JDK-8298530] ZipInputStream.readEnd fails for certain ZIP64 archives with small files - Java Bug System (openjdk.org)
There seem to be unofficial patches. We seem to be bothered by th
On Mon, 29 Apr 2024 14:56:23 GMT, Jan Lahoda wrote:
>> Consider code like:
>>
>> public class ConsoleTest {
>> public static void main(String... args) {
>> System.console().printf("Hello!");
>> }
>> }
>>
>>
>> When run as:
>>
>> $ java ConsoleTest.java >/dev/null
>>
>>
>> it
On Tue, 23 Apr 2024 18:21:30 GMT, Sonia Zaldana Calles
wrote:
> The JNI error message didn’t previously get reported before the regression
> was introduced, so I just wanted to make sure we were okay with this.
I think such errors have a very high potential to confuse the hell out of the
poor
this
particular globe weakness, but I still wonder if that dependency is needed. For
some platforms like AIX it even drags on an additional library dependency. (Not
to mention different charger tables and especially ugly locale dependencies for
containers).
Gruß
Bernd
—
https
Would be helpful to point to the posix standard/requirement which mandates
this. Do you mean a single Unix api description or a posix command spec? I
don’t think I know of any such things.
Wikipedia claims A POSIX-conforming variant of inet_aton, the inet_pton()
function, supports only the four
On Mon, 25 Mar 2024 22:23:03 GMT, Jesper Wilhelmsson
wrote:
>> The problem is that the project is a joined work and has multiple variants
>> of copyrights (see the readme). I don’t think it’s on the safe side to pick
>> a single (non recent) copyright - especially if you change attribution aft
On Mon, 25 Mar 2024 20:34:06 GMT, Kim Barrett wrote:
>> The header file contains more claims
>> https://github.com/veorq/SipHash/blob/master/halfsiphash.h
>
> That header only contains a single function declaration for an entry point
> into the implementation.
> HotSpot doesn't use that functio
On Mon, 25 Mar 2024 20:07:02 GMT, Kim Barrett wrote:
>> src/hotspot/share/legal/siphash.md line 9:
>>
>>> 7:Copyright (c) 2012-2021 Jean-Philippe Aumasson
>>> 8:
>>> 9:Copyright (c) 2012-2014 Daniel J. Bernstein
>>
>> Why would you remove a author or year range?
>
> To match the li
On Fri, 22 Mar 2024 15:18:29 GMT, Jesper Wilhelmsson
wrote:
> Updated and moved the license file.
src/hotspot/share/legal/siphash.md line 9:
> 7:Copyright (c) 2012-2021 Jean-Philippe Aumasson
> 8:
> 9:Copyright (c) 2012-2014 Daniel J. Bernstein
Why would you remove a author or ye
On Mon, 11 Mar 2024 17:46:14 GMT, Chad Rakoczy wrote:
> Fix for [8325621](https://bugs.openjdk.org/browse/JDK-8325621)
>
> Updates jspawnhelper to check that JDK version and jspawnhelper version are
> the same. Updates test to include check for version. Also tested manually by
> replacing jspa
hared libraries,
possibly the class
data archives, and when it comes to multiple OS there are different classes
In the JCL and even templates for JDK resources differ by OS. Some modules are
OS
Specific (like SunMSCAPI provider for 64Bit Windows)
Gruss
Bernd
—
https://bernd.eckenfels.net
On Tue, 13 Feb 2024 17:54:43 GMT, Roger Riggs wrote:
>> Dan Lutker has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove unnecessary var.
>
> test/lib/jdk/test/lib/Platform.java line 145:
>
>> 143: }
>> 144:
>> 145: public sta
On Sat, 16 Dec 2023 17:48:39 GMT, Archie Cobbs wrote:
> If what you're saying is "Previously we were implicitly verifying that the
> data reported by `available()` was actually there, and now we're no longer
> verifying that" then that's not correct.
I mean it verified the non-zero behavior, n
On Fri, 15 Dec 2023 21:13:07 GMT, Archie Cobbs wrote:
>> `GZIPInputStream`, when looking for a concatenated stream, relies on what
>> the underlying `InputStream` says is how many bytes are `available()`. But
>> this is inappropriate because `InputStream.available()` is just an estimate
>> and
On Fri, 8 Dec 2023 21:11:57 GMT, Naoto Sato wrote:
> in recent JDK releases.
ok, could be that this has changed before. It does make sense to not allow it
if the sequence of initialisation can’t be guaranteed.
-
PR Comment: https://git.openjdk.org/jdk/pull/16986#issuecomment-18487
On Sat, 9 Dec 2023 12:09:37 GMT, Markus KARG wrote:
>> @mkarg I guess the method can only be implemented by subclasses residing in
>> the same package with `OutputStream`, right?
>>
>> @AlanBateman fixed
>
> @stsypanov Yes but still it is just weird to ask any output stream if *it* is
> truste
On Wed, 6 Dec 2023 20:55:48 GMT, Naoto Sato wrote:
>> Currently, Locale-related system properties, such as `user.language` or
>> `user.country`, are initialized when the `Locale` class is loaded. Making
>> them static properties is safer than relying on the `Locale` class loading
>> timing, wh
On Fri, 1 Dec 2023 14:05:37 GMT, Bernd wrote:
>> Did you review if all Java.* streams are safe?
>>
>> There are a few stream adapters in sun.nio.ch, which would benefit this
>> optimization too, unfortunately they wrap the arrays with ByteBuffer.wrap, I
>>
On Thu, 30 Nov 2023 09:36:01 GMT, Bernd wrote:
>> Sergey Tsypanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8320971: Trust any OutputStream from java.*
>
> Did you review if all Java.* streams are saf
On Thu, 30 Nov 2023 08:58:18 GMT, Sergey Tsypanov wrote:
>> It looks like we can skip copying of `byte[]` in
>> `BufferedInputStream.implTransferTo()` for `OutputStreams` residing in
>> `java.io`.
>>
>> See comment by @vlsi in
>> https://github.com/openjdk/jdk/pull/10525/files#diff-e19c508d1b
On Wed, 15 Nov 2023 18:30:27 GMT, Justin Lu wrote:
>> src/java.base/share/classes/java/util/zip/ZipFile.java line 498:
>>
>>> 496: */
>>> 497: @Override
>>> 498: public String toString() {
>>
>> I don't think the file name on its own is very helpful as it may not be
>> unique or t
On Wed, 15 Nov 2023 18:33:47 GMT, Justin Lu wrote:
>> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319982)
>> which overrides and provides an implementation of `toString()` in
>> _java.util.zip.ZipFile_ (and by extension, _java.util.jar.JarFile_).
>>
>> This change is p
Or maybe make it more generic and allow to specify the base as well, that’s more along the line of existing toString() methods of numbers.besides convinience, I wonder if it would actually improve performance, how good is the JIT and EA in that case, did you run some bench
Hello,I don’t really understand this discussion,is the normal IO and NIO api not already interpreting error codes? (I am quite sure I often see nationalized error messages in exceptions).https://github.com/openjdk/jdk/blob/76b9011c9ecb8c0c713a58d034f281ba70d65d4e/src/java.
If you return the short buffer (on IOException), does it need to cache the pending exception or can you just rely on the next read hitting that underlying exception again? (Is there an actual guarantee somewhere that the read position is not altered on IOExceptions or that
On Wed, 17 May 2023 12:40:58 GMT, Volker Simonis wrote:
> If we do a short read we will work with corrupted `ChildStuff` and
> `SpawnInfo` structures.
i would agree failing after readFully returns short reads is fine. It won’t
stop the method from retrying/completing endlessly in a pathologica
On Fri, 12 May 2023 15:24:19 GMT, Volker Simonis wrote:
> Since JDK13, executing commands in a sub-process defaults to the so called
> `POSIX_SPAWN` launching mechanism (i.e.
> `-Djdk.lang.Process.launchMechanism=POSIX_SPAWN`) on Linux. This works by
> using `posix_spawn(3)` to firstly start a
Good finding, but just a related comment: it has been customary to request sha1prng, but that practice should be re-evaluated since all new RNG mode features (including the parameters) are absent in that implementation amd it might sooner or later create compliance issues
While we are at it, is Locale.ROOT character types guaranteed to only touch ASCII for case conversion, or could it use latin1 ranges, anyway? (Especially if we think compact strings). How about unassigned or illegal Code units?GrussBernd-- http://bernd.eckenfels.net
On Wed, 5 Apr 2023 19:20:08 GMT, Roger Riggs wrote:
>> Define an internal jdk.internal.util.Architecture enumeration and static
>> methods to replace uses of the system property `os.arch`.
>> The enumeration values are defined to match those used in the build.
>> The initial values are: `X64, X8
If you need order using List instead of Set/Map is perfectly fine decision of your students. It’s good if they learn that early. :)
-- http://bernd.eckenfels.net
Von: core-libs-dev im Auftrag von Remi Forax Gesendet: Sonntag, März 26, 2023 1:42 PMAn: Jens L
The alternative is to specify the charset you want to use in code explicitely and no longer rely on a platform encoding since it made all kinds of problems for being configurable and therefore intentionally removed.GrussBernd-- http://bernd.eckenfels.net
Von: cor
On Wed, 14 Sep 2022 07:33:32 GMT, Сергей Цыпанов wrote:
> > Does that mean there is no explicit test for DataInputStream endianess
>
> @ecki I think this is not strictly necessary as order of bytes is explicitly
> specified in JavaDoc of `DataInput` implemented by `DataInputStream`
Hm, maybe I
On Thu, 25 Aug 2022 09:25:55 GMT, Alan Bateman wrote:
> > What is wrong with the change?
>
> You'll need parentheses to make that work, changing it to use '|' would work
> too.
Does that mean there is no explicit test for DataInputStream endianess (if only
accidential tests checking for magic
BTW, after then ensures it looks like a good candidate for a system-assert for
not-null for all of those fields, right?
Gruss
Bernd
--
http://bernd.eckenfels.net
Von: core-libs-dev im Auftrag von Andrey
Turbanov
Gesendet: Thursday, September 1, 2022 8:24:58 AM
On Fri, 15 Jul 2022 19:33:56 GMT, Ryan Ernst wrote:
>> This commit guards uses of Files methods returning path streams in
>> java.base to use try-with-resources.
>
> Ryan Ernst has updated the pull request incrementally with one additional
> commit since the last revision:
>
> address formatt
Hello Roger,
Thanks for the Tipp, that’s indeed a good reason why it’s not needed in the
legacy SDF API.
Gruss
Bernd
--
http://bernd.eckenfels.net
Von: core-libs-dev im Auftrag von Roger Riggs
Gesendet: Friday, July 15, 2022 4:14:16 PM
An: core-libs-dev
of names like
“June,june,JUNE” or have it allow a special case insensitive configuration
option. (The list would also allow custom short forms like “ja” “fe” but not
“ju”’
Am I missing something?
Gruss
Bernd
--
http://bernd.eckenfels.net
to request the default impl unless you define global
config or system property (and this also required the impl class name so you
can as well create a instance reflectively).
Gruss
Bernd
--
http://bernd.eckenfels.net
I think it means for the zero case it should not use the else part if it has
not the default sizing, since somebody might have created it with a specific
size. Not sure if it matters much either way.
--
http://bernd.eckenfels.net
Von: core-libs-dev im Auftrag v
charset if they are not part of a
platform module (optimized string access)?
Gruss
Bernd
--
http://bernd.eckenfels.net
Von: core-libs-dev im Auftrag von Alan Bateman
Gesendet: Thursday, July 7, 2022 11:50:39 AM
An: build-...@openjdk.org ; core-libs-dev
instance if not needed,
this also applies to replace or escape or similar functions - just don’t rely
on it).
Gruss
Bernd
--
https://urldefense.com/v3/__http://bernd.eckenfels.net__;!!ACWV5N9M2RV99hQ!OCDLUrMtQ8iDuN5ZBlmPgMeJIq1GGLAOuq8CauKNxaqHh-RiIaLCGL-ScrHHeUpLQyeWzm4wj0SMmsGK9Lhbcm5T1Q
67 matches
Mail list logo
