ok2c commented on a change in pull request #234:
URL:
https://github.com/apache/httpcomponents-core/pull/234#discussion_r536771602
##########
File path: httpcore5/src/main/java/org/apache/hc/core5/net/URIBuilder.java
##########
@@ -499,6 +499,16 @@ public URIBuilder setPath(final String path) {
return this;
}
+ /**
+ * Appends path to URI. The value is expected to be unescaped and may
contain non ASCII characters.
+ *
+ * @return this.
+ */
+ public URIBuilder appendPath(final String path) {
Review comment:
@larshelge One last thing. I suppose this should be a little more
efficient by avoiding unnecessary operations and intermediate garbage.
```
public URIBuilder appendPath(final String path) {
if (path != null) {
appendPathSegments(splitPath(path);
}
return this;
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
For additional commands, e-mail: dev-h...@hc.apache.org
