[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8295044: Implementation of Foreign Function and Memory API (Second Preview) [v35]
From:       Alan Bateman <alanb () openjdk ! org>
Date:       2022-11-30 16:41:16
Message-ID: ifYkXDEMXNqgWiktjDfPbS1dFLMkDosUfVrWCzzfobc=.d43429a8-82aa-4439-a6d9-60eafdc5c9b8 () github ! com
[Download RAW message or body]

On Wed, 30 Nov 2022 15:30:40 GMT, Maurizio Cimadamore <mcimadamore@openjdk.org> \
wrote:

> > This PR contains the API and implementation changes for JEP-434 [1]. A more \
> > detailed description of such changes, to avoid repetitions during the review \
> > process, is included as a separate comment. 
> > [1] - https://openjdk.org/jeps/434
> 
> Maurizio Cimadamore 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 67 additional commits since the last \
> revision: 
> - Merge branch 'master' into PR_20
> - Address review comment
> - Polish javadoc:
> * Make sure that first para of class javadoc is succinct and descriptive
> * Remove references to "access" var handle or "memory segment view" var handle \
>                 (just use var handle)
> * Minor tweak to layout classes javadoc - use `@see` in value layouts instead of a \
>                 dedicated para.
> * Other minor typos fixes
> - Address review comments
> - * remove unused Scoped interface
> * re-add trusting of final fields in layout class implementations
> * Fix BulkOps benchmark, which had alignment issues
> - Fix bit vs. byte mismatch in test
> - Fix wrong check in MemorySegment::spliterator/elements
> (The check which ensures that the segment size is multiple of spliterator element \
>                 size is bogus)
> - Address more review comments
> - Fix bad @throws in MemorySegment::copy methods
> - Address review comments
> - ... and 57 more: https://git.openjdk.org/jdk/compare/4c9f206a...8668fb39

src/java.base/share/classes/java/nio/channels/FileChannel.java line 1004:

> 1002:      * Maps a region of this channel's file into a new mapped memory segment, \
>                 with the given offset,
> 1003:      * size and memory session. The {@linkplain MemorySegment#address() \
>                 address} of the returned memory segment
> 1004:      * is the starting address of the mapped off-heap region backing the \
> segment.

Would you mind reflowing this paragraph to that the line lengths are a bit more \
consistent with the paragraphs that follow? That would also help with side-by-side \
views when looking at changes.

-------------

PR: https://git.openjdk.org/jdk/pull/10872


[prev in list] [next in list] [prev in thread] [next in thread]