RFR: 8318598: FFM stylistic cleanups
Alan Bateman
alanb at openjdk.org
Fri Oct 20 16:18:41 UTC 2023
On Fri, 20 Oct 2023 13:35:59 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
> Port over stylistic changes from the panama-foreign repo. The following PRs are included:
>
> - https://github.com/openjdk/panama-foreign/pull/863
> - https://github.com/openjdk/panama-foreign/pull/888
> - https://github.com/openjdk/panama-foreign/pull/889
> - https://github.com/openjdk/panama-foreign/pull/893
> - https://github.com/openjdk/panama-foreign/pull/894
> - https://github.com/openjdk/panama-foreign/pull/897
> - https://github.com/openjdk/panama-foreign/pull/906
>
> Testing: `jdk_foreign`
src/java.base/share/classes/jdk/internal/foreign/AbstractMemorySegmentImpl.java line 635:
> 633: int elementCount) {
> 634:
> 635: var dstInfo = Utils.BaseAndScale.of(dstArray);
This is a case where I personally wouldn't use `var` because RHS is not clear, I have no idea what type dstInfo is.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16286#discussion_r1367209975
More information about the core-libs-dev
mailing list