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
