RFR: 8234335: Remove line break in class declaration in java.base

John Rose john.r.rose at oracle.com
Tue Nov 19 19:17:22 UTC 2019


On Nov 19, 2019, at 4:00 AM, Lance Andersen <lance.andersen at oracle.com> wrote:
> 
> Seems to be a “your milage varies”.  I am fine with whatever the final decision is.  However, I do believe the comment above reads better and aligns the methods better.

FWIW, and as author of many of the lines being changed, I prefer that comment on a separate from the actual modifiers.  I think the basic modifier patterns are easier to recognize quickly when comments and annotations are separate.

— John

P.S. And for the record, my intention of writing non-public is to ensure make it clear to maintainers that the absence of any modifier is both intentional and important (to security, in most cases).  The language doesn’t give any more explicit way to document that default access is required.



More information about the core-libs-dev mailing list