[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:49:00
Message-ID: 1Ao-HvZlCHoGgLIJSJTXOOnvoR1pRtaZoljYUZpFEv0=.1a93bb09-06aa-4fad-905a-41f5f12b6945 () 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/3e822e72...8668fb39

src/java.base/share/classes/java/lang/ModuleLayer.java line 313:

> 311:          * where possible.
> 312:          *
> 313:          * @since 20

We usually put the "@since 20" after the params/return/throws.

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

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


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