andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix memory leak in cache_test introduced in the previous commit

Browse Source 
Test Plan: Verified that valgrind build passes for cache_test

Reviewers: igor

Reviewed By: igor

Subscribers: dhruba

Differential Revision: https://reviews.facebook.net/D37665
This commit is contained in:
Aashish Pant 2015-04-26 21:47:30 -07:00
parent 4961a9622c
commit 3db81d535a
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
util/cache_test.cc
View File

@ -383,6 +383,11 @@ TEST_F(CacheTest, SetCapacity) {
cache->SetCapacity(7); cache->SetCapacity(7);
ASSERT_EQ(7, cache->GetCapacity()); ASSERT_EQ(7, cache->GetCapacity());
ASSERT_EQ(7, cache->GetUsage()); ASSERT_EQ(7, cache->GetUsage());
// release remaining 5 to keep valgrind happy
for (size_t i = 5; i < 10; i++) {
cache->Release(handles[i]);
}
} }
TEST_F(CacheTest, OverCapacity) { TEST_F(CacheTest, OverCapacity) {