On 04/19/2017 02:23 PM, Sergey Bylokhov wrote:
In this grep output I see absolutely the same 5 spaces indentation instead of 4 spaces,
multiple spaces around "=".
The source of those formatting issues maybe the same (replacing tabs by
spaces). And I did not get why the source is important?
Its is unimportant because most of them are not in the public API, some of them
are aligned intentionally, but if you will find something useful feel free to
file it file it and fix.
I see a lot of such formatting issues (hundreds) in public APIs. Are you
proposing to create bugs per each of them?
Why not fix them in this bug? This is very easy and harmless.
From the descripton of the bug you pretend:
"Uncommon formatting and typos in java.desktop module"
Then why only javax.sound is considered? A bug tittle issue?
Did you read the text which were in the first email, which describe what things
were fixed?
Yes, I did. But it doesn't correspond to JIRA's title and description.
But this contradicts to the webrev you've sent because it contains
src/java.desktop/share/classes/java/awt/image/AbstractMultiResolutionImage.java
src/java.desktop/share/classes/java/awt/package-info.java
src/java.desktop/share/classes/module-info.java
files which are outside javax.sound AFAIK.
Just reread an initial message which has 3 points.
Also,
src/java.desktop/share/classes/javax/swing/JComponent.java
3738 * the same functionality and it is handled in {@Component}.
should this mistake be corrected? JComponent is also a public API.
Yes, it should be fixed. This line was added after I created the patch. The new
version is here:
http://cr.openjdk.java.net/~serb/8178971/webrev.01
Thanks. Yet another one:
src/java.desktop/share/classes/javax/swing/text/html/AccessibleHTML.java
1655 * @gsee #getAccessibleChild
I suggest to grep code for javadoc keywords (@...) typos.
--Semyon
