Using raw pointers for manager in Variable.h since weak_ptr::lock() often seems to be a bottle neck during, e.g., model building.

src/storm/storage/expressions/Variable.cpp

@@ -3,11 +3,11 @@
 
 namespace storm {
     namespace expressions {
-        Variable::Variable() {
+        Variable::Variable() : manager(nullptr) {
             // Intentionally left empty.
         }
         
-        Variable::Variable(std::shared_ptr<ExpressionManager const> const& manager, uint_fast64_t index) : manager(manager), index(index) {
+        Variable::Variable(std::shared_ptr<ExpressionManager const> const& manager, uint_fast64_t index) : manager(manager.get()), index(index) {
             // Intentionally left empty.
         }
         
@@ -52,7 +52,8 @@ namespace storm {
         }
         
         ExpressionManager const& Variable::getManager() const {
-            return *manager.lock();
+            STORM_LOG_ASSERT(manager != nullptr, "The variable does not have a manager.");
+            return *manager;
         }
 
         bool Variable::hasBooleanType() const {

src/storm/storage/expressions/Variable.h

@@ -138,7 +138,7 @@ namespace storm {
 
         private:
             // The manager that is responsible for this variable.
-            std::weak_ptr<ExpressionManager const> manager;
+            ExpressionManager const* manager;
             
             // The index of the variable.
             uint_fast64_t index;