[foreign-memaccess] RFR: 8246053: Add a mask for default access modesAdd a default access mode flag
chegar at openjdk.java.net
Thu May 28 14:48:09 UTC 2020
On Thu, 28 May 2020 13:28:45 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
> 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.
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`DEFAULT_ACCESS` and not something like `ALL_MODES`, or `ALL_ACCESS`, or some such?
Marked as reviewed by chegar (no project role).
More information about the panama-dev