< prev index next >
src/java.base/share/classes/sun/net/www/http/HttpClient.java
Print this page
rev 52979 : 8215281: Use String.isEmpty() when applicable in java.base
Reviewed-by: TBD
*** 601,611 ****
// should not be part of the RequestURI. It should be an
// absolute URI which does not have a fragment part.
StringBuilder result = new StringBuilder(128);
result.append(url.getProtocol());
result.append(":");
! if (url.getAuthority() != null && url.getAuthority().length() > 0) {
result.append("//");
result.append(url.getAuthority());
}
if (url.getPath() != null) {
result.append(url.getPath());
--- 601,611 ----
// should not be part of the RequestURI. It should be an
// absolute URI which does not have a fragment part.
StringBuilder result = new StringBuilder(128);
result.append(url.getProtocol());
result.append(":");
! if (url.getAuthority() != null && !url.getAuthority().isEmpty()) {
result.append("//");
result.append(url.getAuthority());
}
if (url.getPath() != null) {
result.append(url.getPath());
*** 617,627 ****
fileName = result.toString();
} else {
fileName = url.getFile();
! if ((fileName == null) || (fileName.length() == 0)) {
fileName = "/";
} else if (fileName.charAt(0) == '?') {
/* HTTP/1.1 spec says in 5.1.2. about Request-URI:
* "Note that the absolute path cannot be empty; if
* none is present in the original URI, it MUST be
--- 617,627 ----
fileName = result.toString();
} else {
fileName = url.getFile();
! if ((fileName == null) || (fileName.isEmpty())) {
fileName = "/";
} else if (fileName.charAt(0) == '?') {
/* HTTP/1.1 spec says in 5.1.2. about Request-URI:
* "Note that the absolute path cannot be empty; if
* none is present in the original URI, it MUST be
< prev index next >