> The implementation of MemorySessionImpl::isAccessibleBy is incorrect, and
> ends up always returning `false` for scopes associated with shared arenas.
> This patch rectifies that, and adds some tests.
Maurizio Cimadamore has updated the pull request incrementally with one
additional commit
On Tue, 17 Jan 2023 18:30:16 GMT, Maurizio Cimadamore
wrote:
> The implementation of MemorySessionImpl::isAccessibleBy is incorrect, and
> ends up always returning `false` for scopes associated with shared arenas.
> This patch rectifies that, and adds some tests.
Marked as reviewed by jvernee
On Tue, 17 Jan 2023 18:30:16 GMT, Maurizio Cimadamore
wrote:
> The implementation of MemorySessionImpl::isAccessibleBy is incorrect, and
> ends up always returning `false` for scopes associated with shared arenas.
> This patch rectifies that, and adds some tests.
Looks okay to me, surprised
The implementation of MemorySessionImpl::isAccessibleBy is incorrect, and ends
up always returning `false` for scopes associated with shared arenas.
This patch rectifies that, and adds some tests.
-
Commit messages:
- Fix bug in MemorySessionImpl::isAccessibleBy
Changes: