[foreign-memaccess] RFR 8226421: Remove LayoutPath API

[Maurizio Cimadamore]

Here's the new patch:

http://cr.openjdk.java.net/~mcimadamore/panama/8226421_v3/

I've significantly cleaned up the javadoc around the layout path-related 
methods and classes and also added a test which is mostly inspired from 
the test that I had already put together for the original 'path 
language' patch.

Maurizio

On 26/06/2019 12:00, Maurizio Cimadamore wrote:
> 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.