8246053: Add a mask for default access modesAdd a default access mode flag #188
Conversation
|
@mcimadamore This change now passes all automated pre-integration checks, type
There are currently no new commits on the
|
This looks good. I've ended up doing similar in tests too, so I think that this is a good addition. I might quibble about the name, why |
@mcimadamore this pull request can not be integrated into git checkout allaccess
git fetch https://git.openjdk.java.net/panama-foreign foreign-memaccess
git merge FETCH_HEAD
# resolve conflicts and follow the instructions given by git merge
git commit -m "Merge foreign-memaccess"
git push |
/integrate |
@mcimadamore |
Looks good. |
This is a simple change which adds a 'default' access mask. This was pointed out during CSR review, and I realized that many tests were also using it.
It can also be handy when creating more restricted masks (e.g. start with default and remove one or two access modes).
I've updated tests, documentation and some of the impl to use the new mask accordingly.
Progress
Issue
Reviewers
Download
$ git fetch https://git.openjdk.java.net/panama-foreign pull/188/head:pull/188
$ git checkout pull/188