Wrap GenericSchemaValidator to destruct the validator first

Tencent · Dec 26, 2023 · 7c4d5f0 · 7c4d5f0
1 parent b3aa1f5
commit 7c4d5f0
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/test/perftest/schematest.cpp b/test/perftest/schematest.cpp
@@ -205,13 +205,14 @@ TEST_F(Schema, TestSuite) {
     for (int i = 0; i < trialCount; i++) {
         for (TestSuiteList::const_iterator itr = testSuites.begin(); itr != testSuites.end(); ++itr) {
             const TestSuite& ts = **itr;
-            GenericSchemaValidator<SchemaDocument, BaseReaderHandler<UTF8<> >, MemoryPoolAllocator<>>  *validator = new GenericSchemaValidator<SchemaDocument, BaseReaderHandler<UTF8<> >, MemoryPoolAllocator<>>(*ts.schema, &validatorAllocator);
-            for (DocumentList::const_iterator testItr = ts.tests.begin(); testItr != ts.tests.end(); ++testItr) {
-                validator->Reset();
-                (*testItr)->Accept(*validator);
-                testCount++;
+            {
+                GenericSchemaValidator<SchemaDocument, BaseReaderHandler<UTF8<> >, MemoryPoolAllocator<> >  validator(*ts.schema, &validatorAllocator);
+                for (DocumentList::const_iterator testItr = ts.tests.begin(); testItr != ts.tests.end(); ++testItr) {
+                    validator->Reset();
+                    (*testItr)->Accept(*validator);
+                    testCount++;
+                }
             }
-            delete validator;
             validatorAllocator.Clear();
         }
     }