+1 > On May 24, 2019, at 9:15 PM, Joe Darcy <[email protected]> wrote: > > Hello, > > Please review the patch below to address: > > JDK-8224783: Javadoc of String strip methods uses link where linkplain > would be better > > Thanks, > > -Joe > > diff -r 4947a097db60 src/java.base/share/classes/java/lang/String.java > --- a/src/java.base/share/classes/java/lang/String.java Fri May 24 > 15:06:58 2019 -0400 > +++ b/src/java.base/share/classes/java/lang/String.java Fri May 24 > 18:14:28 2019 -0700 > @@ -2690,21 +2690,21 @@ > > /** > * Returns a string whose value is this string, with all leading > - * and trailing {@link Character#isWhitespace(int) white space} > + * and trailing {@linkplain Character#isWhitespace(int) white space} > * removed. > * <p> > * If this {@code String} object represents an empty string, > * or if all code points in this string are > - * {@link Character#isWhitespace(int) white space}, then an empty string > + * {@linkplain Character#isWhitespace(int) white space}, then an empty > string > * is returned. > * <p> > * Otherwise, returns a substring of this string beginning with the first > - * code point that is not a {@link Character#isWhitespace(int) white > space} > + * code point that is not a {@linkplain Character#isWhitespace(int) > white space} > * up to and including the last code point that is not a > - * {@link Character#isWhitespace(int) white space}. > + * {@linkplain Character#isWhitespace(int) white space}. > * <p> > * This method may be used to strip > - * {@link Character#isWhitespace(int) white space} from > + * {@linkplain Character#isWhitespace(int) white space} from > * the beginning and end of a string. > * > * @return a string whose value is this string, with all leading > @@ -2722,19 +2722,19 @@ > > /** > * Returns a string whose value is this string, with all leading > - * {@link Character#isWhitespace(int) white space} removed. > + * {@linkplain Character#isWhitespace(int) white space} removed. > * <p> > * If this {@code String} object represents an empty string, > * or if all code points in this string are > - * {@link Character#isWhitespace(int) white space}, then an empty string > + * {@linkplain Character#isWhitespace(int) white space}, then an empty > string > * is returned. > * <p> > * Otherwise, returns a substring of this string beginning with the first > - * code point that is not a {@link Character#isWhitespace(int) white > space} > + * code point that is not a {@linkplain Character#isWhitespace(int) > white space} > * up to and including the last code point of this string. > * <p> > * This method may be used to trim > - * {@link Character#isWhitespace(int) white space} from > + * {@linkplain Character#isWhitespace(int) white space} from > * the beginning of a string. > * > * @return a string whose value is this string, with all leading white > @@ -2752,19 +2752,19 @@ > > /** > * Returns a string whose value is this string, with all trailing > - * {@link Character#isWhitespace(int) white space} removed. > + * {@linkplain Character#isWhitespace(int) white space} removed. > * <p> > * If this {@code String} object represents an empty string, > * or if all characters in this string are > - * {@link Character#isWhitespace(int) white space}, then an empty string > + * {@linkplain Character#isWhitespace(int) white space}, then an empty > string > * is returned. > * <p> > * Otherwise, returns a substring of this string beginning with the first > * code point of this string up to and including the last code point > - * that is not a {@link Character#isWhitespace(int) white space}. > + * that is not a {@linkplain Character#isWhitespace(int) white space}. > * <p> > * This method may be used to trim > - * {@link Character#isWhitespace(int) white space} from > + * {@linkplain Character#isWhitespace(int) white space} from > * the end of a string. > * > * @return a string whose value is this string, with all trailing white > @@ -2782,11 +2782,11 @@ > > /** > * Returns {@code true} if the string is empty or contains only > - * {@link Character#isWhitespace(int) white space} codepoints, > + * {@linkplain Character#isWhitespace(int) white space} codepoints, > * otherwise {@code false}. > * > * @return {@code true} if the string is empty or contains only > - * {@link Character#isWhitespace(int) white space} codepoints, > + * {@linkplain Character#isWhitespace(int) white space} > codepoints, > * otherwise {@code false} > * > * @see Character#isWhitespace(int) >
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 [email protected] <mailto:[email protected]>
