[PATCH] D121720: [Debuginfod] Don't depend on Content-Length.

[Daniel Thornburgh via Phabricator via llvm-commits]

mysterymath added a comment.

In D121720#3383954 <https://reviews.llvm.org/D121720#3383954>, @noajshu wrote:

> On the other hand, since the multithreaded access is not an issue, perhaps it's not worth the trouble to make the `HTTPClient` interface stateless.

I'm inclined to think this way; it seems difficult to a-priori determine the best interface to balance future uses and future backing implementations, given how minimal present footprint is.
Keeping it as simple as possible (without leaking boilerplate to the caller) should help make it easier to change when we need to.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121720/new/

https://reviews.llvm.org/D121720