diff options
| author | Christopher Ferris <cferris@google.com> | 2015-10-14 13:20:10 -0700 |
|---|---|---|
| committer | Christopher Ferris <cferris@google.com> | 2015-10-14 14:03:29 -0700 |
| commit | 634df89db65374a035fdd4a0bc62ec37d295c466 (patch) | |
| tree | b7df301c516714bb23d3c13b89ce857407135367 /libcutils/tests/MemsetTest.cpp | |
| parent | ba0005118b78dc8ff456492bb1e16947ab26be9e (diff) | |
| download | system_core-634df89db65374a035fdd4a0bc62ec37d295c466.tar.gz system_core-634df89db65374a035fdd4a0bc62ec37d295c466.tar.bz2 system_core-634df89db65374a035fdd4a0bc62ec37d295c466.zip | |
Make allocations use unique_ptr.
Change-Id: I4b84b8106faeaeb61847f8f9d13fd6444f474efd
Diffstat (limited to 'libcutils/tests/MemsetTest.cpp')
| -rw-r--r-- | libcutils/tests/MemsetTest.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/libcutils/tests/MemsetTest.cpp b/libcutils/tests/MemsetTest.cpp index 45efc519c..a98485faa 100644 --- a/libcutils/tests/MemsetTest.cpp +++ b/libcutils/tests/MemsetTest.cpp @@ -20,6 +20,8 @@ #include <sys/mman.h> #include <sys/types.h> +#include <memory> + #include <cutils/memory.h> #include <gtest/gtest.h> @@ -127,14 +129,14 @@ void RunMemsetTests(test_e test_type, uint32_t value, int align[][2], size_t num min_incr = 2; value |= value << 16; } - uint32_t* expected_buf = new uint32_t[MAX_TEST_SIZE/sizeof(uint32_t)]; + std::unique_ptr<uint32_t[]> expected_buf(new uint32_t[MAX_TEST_SIZE/sizeof(uint32_t)]); for (size_t i = 0; i < MAX_TEST_SIZE/sizeof(uint32_t); i++) { expected_buf[i] = value; } // Allocate one large buffer with lots of extra space so that we can // guarantee that all possible alignments will fit. - uint8_t *buf = new uint8_t[3*MAX_TEST_SIZE]; + std::unique_ptr<uint8_t[]> buf(new uint8_t[3*MAX_TEST_SIZE]); uint8_t *buf_align; for (size_t i = 0; i < num_aligns; i++) { size_t incr = min_incr; @@ -142,7 +144,7 @@ void RunMemsetTests(test_e test_type, uint32_t value, int align[][2], size_t num incr = GetIncrement(len, min_incr); buf_align = reinterpret_cast<uint8_t*>(GetAlignedPtr( - buf+FENCEPOST_LENGTH, align[i][0], align[i][1])); + buf.get()+FENCEPOST_LENGTH, align[i][0], align[i][1])); SetFencepost(&buf_align[-FENCEPOST_LENGTH]); SetFencepost(&buf_align[len]); @@ -153,15 +155,13 @@ void RunMemsetTests(test_e test_type, uint32_t value, int align[][2], size_t num } else { android_memset32(reinterpret_cast<uint32_t*>(buf_align), value, len); } - ASSERT_EQ(0, memcmp(expected_buf, buf_align, len)) + ASSERT_EQ(0, memcmp(expected_buf.get(), buf_align, len)) << "Failed size " << len << " align " << align[i][0] << " " << align[i][1] << "\n"; VerifyFencepost(&buf_align[-FENCEPOST_LENGTH]); VerifyFencepost(&buf_align[len]); } } - delete expected_buf; - delete buf; } TEST(libcutils, android_memset16_non_zero) { |
