[PATCH] D28515: [Support] - Introduce zlib::toString(zlib::Status)

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 11 07:31:12 PST 2017


grimar added inline comments.


================
Comment at: unittests/Support/CompressionTest.cpp:70
 
+TEST(CompressionTest, ZlibMessages) {
+  EXPECT_EQ("", zlib::toString(zlib::StatusOK));
----------------
mgorny wrote:
> To be honest, I don't think this test takes makes a lot of sense since it's pretty much a duplicate of the function itself. As far as I can see, rather than catching bugs it's going to get people frustrated when they change one of the messages.
That is why I did not add it in inital diff.
I was and still unsure if we should or should not test string values like here.


https://reviews.llvm.org/D28515





More information about the llvm-commits mailing list