[PATCH] D121720: [Debuginfod] Don't depend on Content-Length.
Daniel Thornburgh via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 16 12:02:11 PDT 2022
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
More information about the llvm-commits
mailing list