fix: Deletion of an item still hanging in the container stack no longer causes a segfault.

Author: v-ein

src/dearpygui_commands.h

@@ -2933,10 +2933,10 @@ pop_container_stack(PyObject* self, PyObject* args, PyObject* kwargs)
 		return nullptr;
 	}
 
-	mvAppItem* item = containers.top();
+	PyObject* top = ToPyUUIDOrNone(containers.top().get());
 	containers.pop();
 
-	return ToPyUUIDOrNone(item);
+	return top;
 
 }
 
@@ -2958,7 +2958,7 @@ top_container_stack(PyObject* self, PyObject* args, PyObject* kwargs)
 	mvAppItem* item = nullptr;
 	auto& containers = mvItemRegistry::threadContext.containers;
 	if (!containers.empty())
-		item = containers.top();
+		item = containers.top().get();
 
 	return ToPyUUIDOrNone(item);
 }
@@ -2999,7 +2999,7 @@ push_container_stack(PyObject* self, PyObject* args, PyObject* kwargs)
 
 	mvUUID item = GetIDFromPyObject(itemraw);
 
-	mvAppItem* parent = GetItem((*GContext->itemRegistry), item);
+	auto parent = GetRefItem((*GContext->itemRegistry), item);
 	if (parent)
 	{
 		if (DearPyGui::GetEntityDesciptionFlags(parent->type) & MV_ITEM_DESC_CONTAINER)

src/mvItemRegistry.cpp

@@ -43,7 +43,7 @@ TopParent(mvItemRegistry& registry)
 {
 	auto& containers = mvItemRegistry::threadContext.containers;
     if (!containers.empty())
-        return containers.top();
+        return containers.top().get();
     return nullptr;
 }
 

src/mvItemRegistry.h

@@ -60,7 +60,7 @@ struct mvThreadContext
     mvUUID     lastContainerAdded = 0;
     mvUUID     lastRootAdded = 0;
 
-    std::stack<mvAppItem*>                  containers;      // parent stack, top of stack becomes widget's parent
+    std::stack<std::shared_ptr<mvAppItem>>  containers;      // parent stack, top of stack becomes widget's parent
 
     std::shared_ptr<mvAppItem>              capturedItem = nullptr;
     mvPyObject                              captureCallback = nullptr;