diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-10-17 17:19:43 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-10-17 17:27:08 +0000 |
commit | a13c1f486ea9470a72a5dd564929216370f91951 (patch) | |
tree | 4bb3c9283a3035846ef2691cadcd0d0dcb0487b1 /searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp | |
parent | 169187771dfc974f58238fc19db3b534c4b6c9f4 (diff) |
- Use unique_ptr to hide FuzzyMatcher to make most common SearchContext smaller.
- GC unused stringattribute files.
Diffstat (limited to 'searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp')
-rw-r--r-- | searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp b/searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp index 3fa74b78d2a..9b6187bcc77 100644 --- a/searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp +++ b/searchlib/src/tests/attribute/attributemanager/attributemanager_test.cpp @@ -7,7 +7,6 @@ #include <vespa/searchlib/attribute/configconverter.h> #include <vespa/searchlib/attribute/multinumericattribute.h> #include <vespa/searchlib/attribute/multinumericattribute.hpp> -#include <vespa/searchlib/attribute/stringattribute.h> #include <vespa/vespalib/testkit/testapp.h> #include <vespa/log/log.h> @@ -31,7 +30,7 @@ using TestAttributeBase = MultiValueNumericAttribute< IntegerAttributeTemplate<i class TestAttribute : public TestAttributeBase { public: - TestAttribute(const std::string &name) + explicit TestAttribute(const std::string &name) : TestAttributeBase(name) {} @@ -44,18 +43,17 @@ public: TEST("Test attribute guards") { - AttributeVector::SP vec(new TestAttribute("mvint") ); - TestAttribute * v = static_cast<TestAttribute *> (vec.get()); + auto v = std::make_shared<TestAttribute>("mvint"); EXPECT_EQUAL(v->getGen(), unsigned(0)); EXPECT_EQUAL(v->getRefCount(0), unsigned(0)); EXPECT_EQUAL(v->oldest_used_gen(), unsigned(0)); { - AttributeGuard g0(vec); + AttributeGuard g0(v); EXPECT_EQUAL(v->getGen(), unsigned(0)); EXPECT_EQUAL(v->getRefCount(0), unsigned(1)); EXPECT_EQUAL(v->oldest_used_gen(), unsigned(0)); { - AttributeGuard g1(vec); + AttributeGuard g1(v); EXPECT_EQUAL(v->getGen(), unsigned(0)); EXPECT_EQUAL(v->getRefCount(0), unsigned(2)); EXPECT_EQUAL(v->oldest_used_gen(), unsigned(0)); @@ -72,14 +70,14 @@ TEST("Test attribute guards") EXPECT_EQUAL(v->getRefCount(1), unsigned(0)); EXPECT_EQUAL(v->oldest_used_gen(), unsigned(1)); { - AttributeGuard g0(vec); + AttributeGuard g0(v); EXPECT_EQUAL(v->getGen(), unsigned(1)); EXPECT_EQUAL(v->getRefCount(0), unsigned(0)); EXPECT_EQUAL(v->getRefCount(1), unsigned(1)); EXPECT_EQUAL(v->oldest_used_gen(), unsigned(1)); { v->incGen(); - AttributeGuard g1(vec); + AttributeGuard g1(v); EXPECT_EQUAL(v->getGen(), unsigned(2)); EXPECT_EQUAL(v->getRefCount(0), unsigned(0)); EXPECT_EQUAL(v->getRefCount(1), unsigned(1)); |