RFR [9] 8080422: some docs cleanup for core libs (original) (raw)
Roger Riggs Roger.Riggs at Oracle.com
Mon May 18 14:43:46 UTC 2015
- Previous message: RFR [9] 8080422: some docs cleanup for core libs
- Next message: RFR [9] 8080422: some docs cleanup for core libs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Alexander,
Thanks for these needed cleanups; a few corrections below.
- src/java.base/share/classes/jdk/internal/util/xml/impl/Parser.java:
"The grammar *which *this method can read is" can remove the 'which'
- src/java.base/share/classes/sun/misc/CharacterEncoder.java: 107
The {@code } is confusing, typically @code marks a literal but in this case it is a symbol for the new line character. Perhaps use only "newline" without any special chars, as is used in PrintStream.
src/java.base/share/classes/sun/misc/ExtensionDependency.java: 106 Correct the spelling of attriutes:
* @param jarFile containing the *attriutes *declaring the dependencies
* @param jar containing the *attributes *declaring the dependencies
- src/java.base/share/classes/sun/misc/resources/Messages_de.java: 26
Please use {@code instead of
...
This class represents the
ResourceBundle
- This class represents the
ResourceBundle
- This class represents the
- Ditto all of the files in: src/java.base/share/classes/sun/misc/resources/Messages_* files:
Thanks, Roger
On 5/18/2015 10:08 AM, alexander stepanov wrote:
Please see the updated webrev http://cr.openjdk.java.net/~avstepan/8080422/webrev.01/ - some misprints were fixed as well (not 100% sure if "comparision" should be replaced with "comparison", but the latter looks more suitable).
Thanks, Alexander On 15.05.2015 20:16, alexander stepanov wrote: Hello,
Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8080422/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8080422 Just some HTML markup fix. The affected packages should (probably) not be visible in the new modular system, but nevertheless... Thanks, Alexander
- Previous message: RFR [9] 8080422: some docs cleanup for core libs
- Next message: RFR [9] 8080422: some docs cleanup for core libs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]