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

Noah Shutty via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 15 13:01:39 PDT 2022


noajshu added inline comments.


================
Comment at: llvm/lib/Debuginfod/Debuginfod.cpp:122
+/// file is only created if a 200 OK status is observed.
+class StreamedHTTPResponseHandler final : public HTTPResponseHandler {
+  using CreateStreamFn =
----------------
Just out of curiousity, why use `final`? Is this a style convention or is there some performance impact?


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