[foreign-memaccess] RFR 8226421: Remove LayoutPath API

[Maurizio Cimadamore]

Thanks,

I'll polish this and submit another webrev - I guess we could always go 
down the string-y path at a later stage if we need to.

Maurizio

On 26/06/2019 11:03, Jorn Vernee wrote:
> I like this approach. I think the move to CompoundLayout.Path nicely 
> shows where paths are useful. Compared to the string-y approach, I 
> think it's a bit more robust.
>
> About the patch:
>
> In LayoutPathImpl.java
>   - copyright was removed. I don't think that was intentional?
>   - the 'enclosing' field is unused, so I guess that can be removed.
>   - in sequenceElement(long), I think the check should be `index >= 
> seq.elementsSize().getAsLong()`, i.e. using greater than.
>
> Rest looks good.