8236030: Cleanup use of String.toCharArray
Reviewed-by: ksrini, hannesw, igerasim
This commit is contained in:
parent
dc5728ed95
commit
c670ebb56b
@ -1566,12 +1566,14 @@ public class HtmlDocletWriter {
|
||||
return textCleanup(text, isLast, false);
|
||||
}
|
||||
|
||||
private CharSequence textCleanup(String text, boolean isLast, boolean trimLeader) {
|
||||
if (trimLeader) {
|
||||
text = removeLeadingWhitespace(text);
|
||||
}
|
||||
if (isFirstSentence && isLast) {
|
||||
text = removeTrailingWhitespace(text);
|
||||
private CharSequence textCleanup(String text, boolean isLast, boolean stripLeading) {
|
||||
boolean stripTrailing = isFirstSentence && isLast;
|
||||
if (stripLeading && stripTrailing) {
|
||||
text = text.strip();
|
||||
} else if (stripLeading) {
|
||||
text = text.stripLeading();
|
||||
} else if (stripTrailing) {
|
||||
text = text.stripTrailing();
|
||||
}
|
||||
text = utils.replaceTabs(text);
|
||||
return utils.normalizeNewlines(text);
|
||||
@ -1603,25 +1605,6 @@ public class HtmlDocletWriter {
|
||||
return result;
|
||||
}
|
||||
|
||||
private String removeTrailingWhitespace(String text) {
|
||||
char[] buf = text.toCharArray();
|
||||
for (int i = buf.length - 1; i > 0 ; i--) {
|
||||
if (!Character.isWhitespace(buf[i]))
|
||||
return text.substring(0, i + 1);
|
||||
}
|
||||
return text;
|
||||
}
|
||||
|
||||
private String removeLeadingWhitespace(String text) {
|
||||
char[] buf = text.toCharArray();
|
||||
for (int i = 0; i < buf.length; i++) {
|
||||
if (!Character.isWhitespace(buf[i])) {
|
||||
return text.substring(i);
|
||||
}
|
||||
}
|
||||
return text;
|
||||
}
|
||||
|
||||
/**
|
||||
* Return true if relative links should not be redirected.
|
||||
*
|
||||
|
Loading…
x
Reference in New Issue
Block a user