format

Author: mgeplf

src/neuron/container/pool.hpp

@@ -105,21 +105,21 @@ class Pool {
 template <typename T, bool Mutex>
 T* Pool<T, Mutex>::allocate_pool(long count) {
     T* p{};
-    if(subcount_ > 1){
+    if (subcount_ > 1) {
         p = static_cast<T*>(
             nrn_cacheline_calloc(reinterpret_cast<void**>(&p), count * subcount_, sizeof(T)));
-    }else{
+    } else {
         p = new T[count];
     }
     return p;
 }
 
 template <typename T, bool Mutex>
 void Pool<T, Mutex>::deallocate_pool(T* p) {
-    if(subcount_ > 1){
+    if (subcount_ > 1) {
         free(p);
-    }else{
-        delete []p;
+    } else {
+        delete[] p;
     }
 }
 

test/unit_tests/container/pool.cpp

@@ -39,14 +39,6 @@ TEST_CASE("Pool: basic single-object alloc and free", "[Pool]") {
         REQUIRE(a != b);
     }
 
-    SECTION("hpfree allows reuse") {
-        SimpleItem* a = pool.alloc();
-        pool.hpfree(a);
-        SimpleItem* b = pool.alloc();
-        // After freeing, the same slot can be reused
-        //REQUIRE(b == a);
-    }
-
     SECTION("nget tracks outstanding allocations") {
         REQUIRE(pool.nget() == 0);
         SimpleItem* a = pool.alloc();