On Fri, 8 Jul 2022 06:11:22 GMT, Joe Darcy wrote:
> Initial implementation.
This pull request has now been integrated.
Changeset: 7318b222
Author: Joe Darcy
URL:
https://git.openjdk.org/jdk/commit/7318b22209a83c593176ec600647a9b050362932
Stats: 703 lines in 3 files changed:
> Initial implementation.
Joe Darcy has updated the pull request incrementally with two additional
commits since the last revision:
- Adopt alternative implementaiton from review.
- Refactor equivalent() implementation.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/9
> Initial implementation.
Joe Darcy has updated the pull request incrementally with one additional commit
since the last revision:
Implement additional review feedback.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/9422/files
- new: https://git.openjdk.org/jdk/pull/9
> Initial implementation.
Joe Darcy 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 merge/rebase. The pull request contains 23 additional commits since the
last revision:
- Implem
On Sat, 23 Jul 2022 18:00:12 GMT, Raffaello Giulietti wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Implement review feedback.
>
> src/java.base/share/classes/java/lang/Fl
On Sat, 23 Jul 2022 16:38:34 GMT, Raffaello Giulietti wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Implement review feedback.
>
> src/java.base/share/classes/java/lang/Float.jav
> Initial implementation.
Joe Darcy has updated the pull request incrementally with one additional commit
since the last revision:
Implement review feedback.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/9422/files
- new: https://git.openjdk.org/jdk/pull/9422/fi
On Thu, 21 Jul 2022 21:22:19 GMT, Joe Darcy wrote:
>> Initial implementation.
>
> Joe Darcy 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 merge/rebase. The pull r
> Initial implementation.
Joe Darcy 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 merge/rebase. The pull request contains 20 additional commits since the
last revision:
- Method ren
On Mon, 18 Jul 2022 20:30:21 GMT, Joe Darcy wrote:
>> Initial implementation.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Add NaN test.
I added NaN round-trip tests is a separate file since the prop
> Initial implementation.
Joe Darcy has updated the pull request incrementally with one additional commit
since the last revision:
Add NaN test.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/9422/files
- new: https://git.openjdk.org/jdk/pull/9422/files/420a9
> Initial implementation.
Joe Darcy has updated the pull request incrementally with one additional commit
since the last revision:
Preserve NaN sign bit.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/9422/files
- new: https://git.openjdk.org/jdk/pull/9422/files/b09fd
> Initial implementation.
Joe Darcy 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 merge/rebase. The pull request contains 15 additional commits since the
last revision:
- Resp
On Wed, 13 Jul 2022 13:11:33 GMT, Raffaello Giulietti wrote:
>> Initial implementation.
>
> src/java.base/share/classes/java/lang/Float.java line 1174:
>
>> 1172:
>> 1173: short result = 0;
>> 1174: result = (short)(((exp + 15) << 10) | signif_bits);
>
> result =
On Wed, 13 Jul 2022 12:28:29 GMT, Raffaello Giulietti wrote:
>> Initial implementation.
>
> src/java.base/share/classes/java/lang/Float.java line 1090:
>
>> 1088: public static short floatToBinary16AsShortBits(float f) {
>> 1089: if (Float.isNaN(f)) {
>> 1090: //
On Wed, 13 Jul 2022 14:33:49 GMT, Raffaello Giulietti wrote:
>> Initial implementation.
>
> test/jdk/java/lang/Float/SixteenBitFormats.java line 239:
>
>> 237: public static boolean isNaN(short binary16) {
>> 238: return ((binary16 & 0x7c00) == 0x7c00) // Max exponent
>>
On Wed, 13 Jul 2022 12:38:23 GMT, Raffaello Giulietti wrote:
>> Initial implementation.
>
> src/java.base/share/classes/java/lang/Float.java line 1100:
>
>> 1098:
>> 1099: // The overflow threshold is binary16 MAX_VALUE + 1/2 ulp
>> 1100: if (abs_f > (65504.0f + 16.0f) ) {
>
>
On Thu, 14 Jul 2022 20:54:59 GMT, Joe Darcy wrote:
>> src/java.base/share/classes/java/lang/Float.java line 1100:
>>
>>> 1098:
>>> 1099: // The overflow threshold is binary16 MAX_VALUE + 1/2 ulp
>>> 1100: if (abs_f > (65504.0f + 16.0f)
On Mon, 11 Jul 2022 20:31:04 GMT, Mandy Chung wrote:
> This `ClassFileFormatVersion` API is a good proposal. It can also reference
> `java.class.version` system property to map to the latest class file format
> version.
Good idea Mandy; I'll add that reference and start working on the
On Wed, 22 Jun 2022 12:43:53 GMT, Quan Anh Mai wrote:
> Improve the implementation of `Double/Float.isInfinite` to reduce branching.
> Using `>` comparison with `MAX_VALUE` instead of `==` with
> `POSITIVE_INFINITY` improves code emission on x86 and produces similar code
> for arm. This is
On Wed, 13 Jul 2022 15:32:18 GMT, Сергей Цыпанов wrote:
>> `BigInteger.powerCache` is volatile and should be assigned only once in
>> static initializer.
>
> Сергей Цыпанов has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8290079: Rename
On Fri, 8 Jul 2022 06:11:22 GMT, Joe Darcy wrote:
> Initial implementation.
Some notes in the current implementation in the PR: the impetus for the change
is to provide a minimal level of platform support for the binary16
floating-point format defined in IEEE 754. This is done by added
On Wed, 6 Jul 2022 20:49:25 GMT, John R Rose wrote:
> The code examples in these files were exdented to make it easier to extract
> example code and test it, and to maintain equivalence during active
> development of the APIs.
>
> The examples in runnable form are in
>
On Wed, 6 Jul 2022 00:08:04 GMT, liach wrote:
>> Update existing examples in java.lang.invoke.{MethodHandle, MethodHandles}
>> to use snippets rather than the older markup idiom.
>
> src/java.base/share/classes/java/lang/invoke/MethodHandle.java line 272:
>
>> 270: * Here are some examples
Update existing examples in java.lang.invoke.{MethodHandle, MethodHandles} to
use snippets rather than the older markup idiom.
-
Commit messages:
- JDK-8289775: Update java.lang.invoke.MethodHandle[s] to use snippets
Changes: https://git.openjdk.org/jdk/pull/9385/files
Webrev:
On Mon, 4 Jul 2022 06:57:12 GMT, ScientificWare wrote:
> - Correct a deprecated code.
> - Update Copyright.
> - More wide question : How are you dealing with deprecated codes in snippets ?
> ```
> @deprecated(since="9")
> public Integer(int value)
> ```
Looks fine.
There is as of yet
On Wed, 29 Jun 2022 16:51:21 GMT, Roger Riggs wrote:
> I'm thinking of the case where I'm reading class file bytes and pull out the
> major classfile version and want to map it to the enum. If the semantic was
> the "earliest version" supporting the major version then it would be
>
> JDK-8289106: Add model of class file versions to core reflection
Joe Darcy 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 merge/rebase. The pull request contains 10 additional comm
On Tue, 28 Jun 2022 10:32:31 GMT, Raffaello Giulietti wrote:
>> `BigDecimal.morePoint[Left|Right]()` should return the target `this` when
>> the argument is 0 _and_ the scale is non-negative.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since
On Tue, 28 Jun 2022 10:32:31 GMT, Raffaello Giulietti wrote:
>> `BigDecimal.morePoint[Left|Right]()` should return the target `this` when
>> the argument is 0 _and_ the scale is non-negative.
>
> Raffaello Giulietti has updated the pull request incrementally with one
> additional commit since
On Wed, 29 Jun 2022 14:42:19 GMT, Roger Riggs wrote:
> A static method to map from classfile version (as in the JVMS) to
> ClassFileVersion enum will be useful too.
Just to confirm, you're asking for a method that would map 63 to RELEASE_19, 64
to RELEASE_20, etc.?
Seems reasonable, although
On Mon, 27 Jun 2022 20:26:52 GMT, Joe Darcy wrote:
> JDK-8289106: Add model of class file versions to core reflection
Sending this enum to model class file format versions out for some review
comments before starting to write the tests and CSR, etc.
The class file format has a rich struct
JDK-8289106: Add model of class file versions to core reflection
-
Commit messages:
- Update phrasing.
- Augment and correct docs.
- Correct major versions; RELEASE_0 and RELEASE_1 are both 45.
- Make major method public.
- Add AccessFlag.locations(ClassFileFormatVersion)
- Add
On Fri, 11 Feb 2022 18:31:57 GMT, Joe Darcy wrote:
> This is an early review of changes to better model JVM access flags, that is
> "modifiers" like public, protected, etc. but explicitly at a VM level.
>
> Language level modifiers and JVM level access fla
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
On Wed, 8 Jun 2022 13:12:09 GMT, Raffaello Giulietti wrote:
> These 19'545 doubles were generated on purpose by Paul Zimmermann of INRIA as
> hard test cases.
Marked as reviewed by darcy (Reviewer).
-
PR: https://git.openjdk.org/jdk/pull/9084
On Fri, 17 Jun 2022 16:44:30 GMT, Raffaello Giulietti wrote:
> Extend delegation by instance returned by Random.from() to all methods
> exposed by RandomGenerator.
Looks fine. I agree this implementation is more in keeping with the intent of
the specification.
-
Marked as
On Thu, 23 Jun 2022 14:37:42 GMT, Roger Riggs wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove implSpec tag from Executable.accessFlags since the class is sealed.
>
> src/jav
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
On Wed, 22 Jun 2022 17:33:25 GMT, Mandy Chung wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove implSpec tag from Executable.accessFlags since the class is sealed.
>
> src/jav
On Tue, 14 Jun 2022 14:22:29 GMT, Roger Riggs wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Respond to review feedback.
>
> src/java.base/share/classes/java/lang/reflect/Acces
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
On Fri, 29 Apr 2022 18:49:26 GMT, Mandy Chung wrote:
>> Joe Darcy 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 merge/rebase. The pull request contains 26
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
On Thu, 16 Jun 2022 21:21:36 GMT, Joe Darcy wrote:
> Whatever the motivation for how this method was coded when added in JDK 8,
> since Executable is now a sealed class with Constructor and Method the only
> allowed subclasses, getParameterCount can be coded as an normal abstract
nt in Method and Constructor
> use fields private to each of those classes so the code cannot easily be
> shared in Executable).
>
> Please also review the small accompanying CSR:
> https://bugs.openjdk.org/browse/JDK-8288630
Joe Darcy has updated the pull request incrementally with o
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
On Thu, 16 Jun 2022 08:21:49 GMT, ExE Boss wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Respond to review feedback.
>
> src/java.base/share/classes/java/lang/reflect/Member.java line
Whatever the motivation for how this method was coded when added in JDK 8,
since Executable is now a sealed class with Constructor and Method the only
allowed subclasses, getParameterCount can be coded as an normal abstract
method. (The implementation of getParameterCount in Method and
On Thu, 16 Jun 2022 06:27:59 GMT, Joe Darcy wrote:
>> This is an early review of changes to better model JVM access flags, that is
>> "modifiers" like public, protected, etc. but explicitly at a VM level.
>>
>> Language level modifiers and JVM leve
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
uced.
>
> This PR in its current form is meant to give the overall shape of the API. It
> is missing implementations to map from, say, method modifiers to access
> flags, taking into account overlaps in bit positions.
>
> The CSR https://bugs.openjdk.java.net/browse/JDK-8281660 will be fill
701 - 754 of 754 matches
Mail list logo