larshelge commented on a change in pull request #234:
URL:
https://github.com/apache/httpcomponents-core/pull/234#discussion_r536763113
##########
File path: httpcore5/src/main/java/org/apache/hc/core5/net/URIBuilder.java
##########
@@ -499,6 +499,18 @@ 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) {
+ final List<String> segments = new ArrayList<>(getPathSegments());
+ segments.addAll(path != null ? splitPath(path) : new
ArrayList<String>());
+ setPathSegments(segments);
Review comment:
I fully agree. I thought about it but favored consistency with existing
code, e.g. `setPathSegments(String...)` and `setPathSegments(List)`. I am happy
to change it.
----------------------------------------------------------------
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
