Fix memory leak in unit test of Bitcode/BitReaderTest.cpp
authorDerek Schuff <dschuff@google.com>
Mon, 3 Aug 2015 21:23:51 +0000 (21:23 +0000)
committerDerek Schuff <dschuff@google.com>
Mon, 3 Aug 2015 21:23:51 +0000 (21:23 +0000)
Fixes obvious memory leak in test
TestForEofAfterReadFailureOnDataStreamer.  Also removes constexpr use
from same test.

Patch by Karl Schimpf.

Differential Revision: http://reviews.llvm.org/D11735

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@243904 91177308-0d34-0410-b5e6-96231b3b80d8

unittests/Bitcode/BitReaderTest.cpp

index d9e207e9ba3b856e2e9d48fc2b54d36b57ffc660..055850da505ac4c5ea7bb486b67f27ab2bb8b733 100644 (file)
@@ -118,10 +118,12 @@ TEST(BitReaderTest, TestForEofAfterReadFailureOnDataStreamer) {
   // Jump to two bytes before end of stream.
   Cursor.JumpToBit((InputSize - 4) * CHAR_BIT);
   // Try to read 4 bytes when only 2 are present, resulting in error value 0.
-  constexpr size_t ReadErrorValue = 0;
+  const size_t ReadErrorValue = 0;
   EXPECT_EQ(ReadErrorValue, Cursor.Read(32));
   // Should be at eof now.
   EXPECT_TRUE(Cursor.AtEndOfStream());
+
+  delete[] Text;
 }
 
 TEST(BitReaderTest, MateralizeForwardRefWithStream) {