[PATCH] D112751: [llvm] [Support] Add HTTP Client Support library.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/unittests/Support/HTTPClient.cpp:75
+
+  EXPECT_THAT_ERROR(Handler.handleHeaderLine("Content-Length: \0\0\0"),
+                    Succeeded());
----------------
This probably doesn't test what it's intended to test - the string will terminate at the first '\0' - if you want to test embedded null characters, you'd need to specify an explicit length when creating the StringRef explicitly, rather than relying on StringRef(const char*) conversion.


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

https://reviews.llvm.org/D112751