RFR [14] 8236077: Clean up the use of modifiers and semicolons

Pavel Rappo pavel.rappo at oracle.com
Tue Dec 17 12:09:54 UTC 2019


Hello,

Please review the following trivial change for https://bugs.openjdk.java.net/browse/JDK-8236077:

    http://cr.openjdk.java.net/~prappo/8236077/webrev.00/

This is a cleanup task. The above change removes the redundant modifiers (e.g. "public abstract" for
interface methods). This allows to recover some precious method-signature-line space, reduce visual
noise, and fix inconsistencies (e.g. where only some of those implicit modifiers were used).
The remaining modifiers are sorted according to the convention [1]. The change does not alter the
indentation or param grouping style in the affected methods, unless impractical.

The change also fixes typos in the javadoc comments, comments, variables' and properties' names. 
Not only does this address aesthetic issues, it also helps with searches.

I suggest reviewing this change using a diff tool with a character-level resolution. 
That is, a tool capable of highlighting mismatching characters in lines that differ.

All test/langtools/jdk/javadoc/doclet tests pass. Copyright years will be fixed before the push.

Thanks,
-Pavel

---------------------------------------------------------------------------------------------------
[1] A related task https://bugs.openjdk.java.net/browse/JDK-8136583



More information about the javadoc-dev mailing list