RFR: 8319556: Harmonize interface formatting in the FFM API [v2]

Per Minborg pminborg at openjdk.org
Tue Nov 7 15:14:31 UTC 2023


On Tue, 7 Nov 2023 11:41:06 GMT, Per Minborg <pminborg at openjdk.org> wrote:

>> This PR proposes to remove two `permits` declarations where the line overflows the stipulated maximum column with. Also, it proposes to harmonize the layout of `permit` formatting so they are the the same throughout the API.
>> 
>> This PR might be perceived as over worked but I think it nice to get consistency across the API now that we go final.
>
> Per Minborg has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Remove redundandt space
>  - Use permits reformatting only when needed

Hrmm. I reformatted all classes of the same category where at least one of them needed reformatting to make it more consistent. For example: `OfBoolean` overflows but not `OfByte`. So, all of the of* were consistently reformatted. So, let's pick a choice here:

1) Reformat only those specific classes needing it not bothering if similar classes look different
2) Keep it as it is.

I am happy to do either.

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

PR Comment: https://git.openjdk.org/jdk/pull/16528#issuecomment-1798832942


More information about the core-libs-dev mailing list