On Thu, 15 Feb 2024 12:19:31 GMT, Magnus Ihse Bursie wrote:
> Since jcheck only checks file in a commit, there is a possibility of us
> getting files in the repository that would not be accepted by jcheck. This
> can happen when extending the set of files checked by jcheck, or if jcheck
> chan
On Thu, 15 Feb 2024 12:19:31 GMT, Magnus Ihse Bursie wrote:
> Since jcheck only checks file in a commit, there is a possibility of us
> getting files in the repository that would not be accepted by jcheck. This
> can happen when extending the set of files checked by jcheck, or if jcheck
> chan
> During analysing a customer case I figured out that we have an inconsistency
> between documentation and actual behavior in class
> com.sun.jndi.ldap.Connection. The [method documentation of
> com.sun.jndi.ldap.Connection::createSocket](https://github.com/openjdk/jdk/blob/3ebe6c192a5dd5cc46ae2
Vote: yes
On 2/13/24 12:25 PM, Brian Burkhalter wrote:
I hereby nominate Raffaello Giulietti to Membership in the Core Libraries Group.
Raffaello has been working in the Core Library team at Oracle since April,
2022. He has authored more than 50 contributions to OpenJDK in a number of
areas i
https://openjdk.org/jeps/466
Summary: Provide a standard API for parsing, generating, and
transforming Java class files. This is a preview API.
- Mark
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 only does this lead to faulty formatting results, but breaks the
ChoiceF
On Thu, 15 Feb 2024 18:54:46 GMT, Brian Burkhalter wrote:
> Revert `@replace` snippet annotation construct to value of `replacement`
> attribute.
LGTM
-
Marked as reviewed by jlu (Committer).
PR Review: https://git.openjdk.org/jdk/pull/17882#pullrequestreview-1883649981
On Thu, 15 Feb 2024 18:54:46 GMT, Brian Burkhalter wrote:
> Revert `@replace` snippet annotation construct to value of `replacement`
> attribute.
I saw the internal discussion and agree with the changes.
-
Marked as reviewed by naoto (Reviewer).
PR Review: https://git.openjdk.org
Revert `@replace` snippet annotation construct to value of `replacement`
attribute.
-
Commit messages:
- 8325990: Remove use of snippet @replace annotation in java.base
Changes: https://git.openjdk.org/jdk/pull/17882/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17882&ran
On Thu, 15 Feb 2024 17:28:52 GMT, Andy Goryachev wrote:
>> Please do not replace those tabs with spaces as they are intentional for
>> testing the runtime to safely ignore them. I suggest replacing them with
>> Unicode escapes (`\u000b`)
>
> `\u000b` is VT (vertical tab)
> `\u0009` or `\t`
On Thu, 15 Feb 2024 17:09:17 GMT, Naoto Sato wrote:
>> All the java/util/Currency tests pass. I also searched the code for "ZZ" but
>> could not find any references in the test.
>
> Please do not replace those tabs with spaces as they are intentional for
> testing the runtime to safely ignore t
On Thu, 15 Feb 2024 15:48:38 GMT, Magnus Ihse Bursie wrote:
>> This looks weird indeed. Luckily it's just test code. Did you run the test
>> after this change?
>
> All the java/util/Currency tests pass. I also searched the code for "ZZ" but
> could not find any references in the test.
Please d
In this PR I updated the tests to use the newer
ProcessTools.createTestJavaProcessBuilder methods to pass VM options to child
processes.
-
Commit messages:
- 8319648: java/lang/SecurityManager tests ignore vm flags
Changes: https://git.openjdk.org/jdk/pull/17878/files
Webrev: htt
On Thu, 15 Feb 2024 14:01:46 GMT, Erik Joelsson wrote:
>> test/jdk/java/util/Currency/currency.properties line 18:
>>
>>> 16: SB=EUR,111,2, 2099-01-01T00:00:00
>>> 17: US=euR,978,2,2001-01-01T00:00:00
>>> 18: ZZ = ZZZ , 999 , 3
>>
>> This looks weird, but so did t
On Thu, 15 Feb 2024 15:11:15 GMT, Christoph Langer wrote:
>> During analysing a customer case I figured out that we have an inconsistency
>> between documentation and actual behavior in class
>> com.sun.jndi.ldap.Connection. The [method documentation of
>> com.sun.jndi.ldap.Connection::createS
> During analysing a customer case I figured out that we have an inconsistency
> between documentation and actual behavior in class
> com.sun.jndi.ldap.Connection. The [method documentation of
> com.sun.jndi.ldap.Connection::createSocket](https://github.com/openjdk/jdk/blob/3ebe6c192a5dd5cc46ae2
On Thu, 15 Feb 2024 12:26:11 GMT, Magnus Ihse Bursie wrote:
>> Since jcheck only checks file in a commit, there is a possibility of us
>> getting files in the repository that would not be accepted by jcheck. This
>> can happen when extending the set of files checked by jcheck, or if jcheck
>>
> Set `interrupted` in `Thread::interrupt` before reading `nioBlocker` for
> correct (Dekker scheme) synchronization with concurrent execution of
> [`AbstractInterruptibleChannel::begin`](https://github.com/openjdk/jdk/blob/59062402b9c5ed5612a13c1c40eb22cf1b97c41a/src/java.base/share/classes/java
> Adding a fast-path for ASCII-only modified UTF-8 strings deserialied via
> Data- and ObjectInputStream
>
> Testing: tier1-3
Claes Redestad has updated the pull request incrementally with one additional
commit since the last revision:
Update src/java.base/share/classes/java/io/DataInputStre
On Thu, 15 Feb 2024 12:19:31 GMT, Magnus Ihse Bursie wrote:
> Since jcheck only checks file in a commit, there is a possibility of us
> getting files in the repository that would not be accepted by jcheck. This
> can happen when extending the set of files checked by jcheck, or if jcheck
> chan
Since jcheck only checks file in a commit, there is a possibility of us getting
files in the repository that would not be accepted by jcheck. This can happen
when extending the set of files checked by jcheck, or if jcheck changes how it
checks files (perhaps due to bug fixes).
I have now run jc
On Thu, 15 Feb 2024 10:33:07 GMT, Claes Redestad wrote:
>> Adding a fast-path for ASCII-only modified UTF-8 strings deserialied via
>> Data- and ObjectInputStream
>>
>> Testing: tier1-3
>
> Claes Redestad has updated the pull request incrementally with one additional
> commit since the last re
On Thu, 15 Feb 2024 10:55:38 GMT, Raffaello Giulietti
wrote:
>> The specification is somewhat ambiguous:
>> https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/DataInput.html#readUTF()
>>
>> There's a sweeping `Throws UTFDataFormatException - if the bytes do not
>> represent a
On Wed, 14 Feb 2024 14:30:02 GMT, Claes Redestad wrote:
>> Ah OK.
>>
>> I didn't check the current code, only the proposed one.
>> Although the specification clearly states that the method should throw, if
>> the current code does not throw on zeros, then it makes sense that the
>> proposed on
On Wed, 14 Feb 2024 20:04:42 GMT, Brian Burkhalter wrote:
> As there are no regression tests added by this request, I assume that
> existing tests must sufficiently cover this area. If so, however, the issue
> has no `noreg-` label.
At least for `ObjectInputStream` there's a large and seemingl
On Thu, 15 Feb 2024 10:33:07 GMT, Claes Redestad wrote:
>> Adding a fast-path for ASCII-only modified UTF-8 strings deserialied via
>> Data- and ObjectInputStream
>>
>> Testing: tier1-3
>
> Claes Redestad has updated the pull request incrementally with one additional
> commit since the last re
> Adding a fast-path for ASCII-only modified UTF-8 strings deserialied via
> Data- and ObjectInputStream
>
> Testing: tier1-3
Claes Redestad has updated the pull request incrementally with one additional
commit since the last revision:
Revert spurious formatting changes
-
Chang
> Adding a fast-path for ASCII-only modified UTF-8 strings deserialied via
> Data- and ObjectInputStream
>
> Testing: tier1-3
Claes Redestad has updated the pull request incrementally with one additional
commit since the last revision:
Revert changes to DataInputStream
-
Change
On Thu, 15 Feb 2024 06:29:24 GMT, Christian Stein wrote:
>> Please review this PR that makes the launcher helper keep a reference to the
>> executable JAR file active after extracting the name of the main class and
>> returning it as Class instance. Now, when loading classes from the JAR file,
On Wed, 14 Feb 2024 15:42:48 GMT, Christian Stein wrote:
> `String jarname` is filled by the caller with the value of `String what`, and
> therefore contains the entire path to the executable JAR file. Using only
> `jarFile.getName()` will prevent any parent directories from being noted in
> e
30 matches
Mail list logo
