[foreign-memaccess+abi] RFR: 8292037: Consider moving Linker::methodType as FunctionDescriptor::toMethodType [v3]

[Maurizio Cimadamore]

On Tue, 13 Sep 2022 15:36:54 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> src/java.base/share/classes/java/lang/foreign/FunctionDescriptor.java line 110:
>> 
>>> 108:      * @return the method type consisting of the carrier types of the layouts in this function descriptor
>>> 109:      * @throws IllegalArgumentException if one or more layouts in the function descriptor can not be mapped to carrier
>>> 110:      *                                  types (e.g. if they are sequence layouts or padding layouts).
>> 
>> Suggestion:
>> 
>>      *                                  types (e.g. if they are padding layouts).
>
> I've removed the support for SequenceLayout now, so I think this is currently correct?

Ah - of course. I didn't notice the bullet being rewritten. I think this is a good move, and one that still give us the option to change things later, if needed (it's always relatively ok to turn throwing cases into non-throwing ones).

-------------

PR: https://git.openjdk.org/panama-foreign/pull/717