rschmitt commented on a change in pull request #106: Immutable HttpEntity classes URL: https://github.com/apache/httpcomponents-core/pull/106#discussion_r256969017
########## File path: httpcore5/src/main/java/org/apache/hc/core5/http/io/entity/InputStreamEntity.java ########## @@ -31,87 +31,52 @@ import java.io.InputStream; import java.io.OutputStream; +import org.apache.hc.core5.annotation.Contract; +import org.apache.hc.core5.annotation.ThreadingBehavior; import org.apache.hc.core5.http.ContentType; import org.apache.hc.core5.util.Args; /** - * A streamed, non-repeatable entity that obtains its content from - * an {@link InputStream}. + * A streamed, non-repeatable entity that obtains its content from an {@link InputStream}. * * @since 4.0 */ +@Contract(threading = ThreadingBehavior.IMMUTABLE_CONDITIONAL) Review comment: I don't really see how an `InputStream` wrapper could be regarded as immutable, unless `IMMUTABLE_CONDITIONAL` specifically means "immutable" in the Java Memory Model sense (all final fields, not made visible to other threads during construction), which has significantly weaker guarantees than true immutability. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org