Add a way to pass in a preconstructed value to ManagedStatic (#2175)

A lot of our cases don't need the lazy construction, but do need manual destruction.
This commit is contained in:
Thad House
2019-12-20 14:06:22 -08:00
committed by Peter Johnson
parent 5e08bb28f8
commit 9cb69c5b46
3 changed files with 83 additions and 0 deletions

View File

@@ -30,6 +30,22 @@ static wpi::mutex* getManagedStaticMutex() {
return ManagedStaticMutex;
}
void ManagedStaticBase::RegisterManagedStatic(void* created,
void (*Deleter)(void*)) const {
std::scoped_lock Lock(*getManagedStaticMutex());
if (!Ptr.load(std::memory_order_relaxed)) {
void *Tmp = created;
Ptr.store(Tmp, std::memory_order_release);
DeleterFn = Deleter;
// Add to list of managed statics.
Next = StaticList;
StaticList = this;
}
}
void ManagedStaticBase::RegisterManagedStatic(void *(*Creator)(),
void (*Deleter)(void*)) const {
assert(Creator);

View File

@@ -43,6 +43,7 @@ protected:
mutable const ManagedStaticBase *Next;
void RegisterManagedStatic(void *(*creator)(), void (*deleter)(void*)) const;
void RegisterManagedStatic(void *created, void (*deleter)(void*)) const;
public:
/// isConstructed - Return true if this object has not been created yet.
@@ -60,6 +61,12 @@ template <class C, class Creator = object_creator<C>,
class Deleter = object_deleter<C>>
class ManagedStatic : public ManagedStaticBase {
public:
ManagedStatic() = default;
ManagedStatic(C* created, void(*deleter)(void*)) {
RegisterManagedStatic(created, deleter);
}
// Accessors.
C &operator*() {
void *Tmp = Ptr.load(std::memory_order_acquire);

View File

@@ -0,0 +1,60 @@
/*----------------------------------------------------------------------------*/
/* Copyright (c) 2019 FIRST. All Rights Reserved. */
/* Open Source Software - may be modified and shared by FRC teams. The code */
/* must be accompanied by the FIRST BSD license file in the root directory of */
/* the project. */
/*----------------------------------------------------------------------------*/
#include "wpi/ManagedStatic.h" // NOLINT(build/include_order)
#include "gtest/gtest.h"
static int refCount = 0;
struct StaticTestClass {
StaticTestClass() { refCount++; }
~StaticTestClass() { refCount--; }
void Func() {}
};
namespace wpi {
TEST(ManagedStaticTest, LazyDoesNotInitialize) {
{
refCount = 0;
wpi::ManagedStatic<StaticTestClass> managedStatic;
ASSERT_EQ(refCount, 0);
}
ASSERT_EQ(refCount, 0);
wpi_shutdown();
}
TEST(ManagedStaticTest, LazyInitDoesntDestruct) {
{
refCount = 0;
wpi::ManagedStatic<StaticTestClass> managedStatic;
ASSERT_EQ(refCount, 0);
managedStatic->Func();
ASSERT_EQ(refCount, 1);
}
ASSERT_EQ(refCount, 1);
wpi_shutdown();
ASSERT_EQ(refCount, 0);
}
TEST(ManagedStaticTest, EagerInit) {
{
refCount = 0;
StaticTestClass* test = new StaticTestClass{};
ASSERT_EQ(refCount, 1);
wpi::ManagedStatic<StaticTestClass> managedStatic(
test, [](void* val) { delete static_cast<StaticTestClass*>(val); });
ASSERT_EQ(refCount, 1);
managedStatic->Func();
ASSERT_EQ(refCount, 1);
}
ASSERT_EQ(refCount, 1);
wpi_shutdown();
ASSERT_EQ(refCount, 0);
}
} // namespace wpi