diff --git a/src/mvItemRegistry.cpp b/src/mvItemRegistry.cpp
index b9feb07cc..2bca82d58 100644
--- a/src/mvItemRegistry.cpp
+++ b/src/mvItemRegistry.cpp
@@ -385,7 +385,7 @@ MoveItemUp(mvItemRegistry& registry, mvUUID uuid)
     {
         for (auto& childset : parent->childslots)
         {
-            for (auto it = childset.begin(), prev_it = childset.end(); it != childset.end(); ++it)
+            for (auto it = childset.begin(), prev_it = childset.begin(); it != childset.end(); ++it)
             {
                 if (it->get() == item)
                 {
diff --git a/src/mvLayoutWindow.cpp b/src/mvLayoutWindow.cpp
index 59261b7ba..1e510f0e9 100644
--- a/src/mvLayoutWindow.cpp
+++ b/src/mvLayoutWindow.cpp
@@ -34,7 +34,7 @@ DebugItem(const char* label, bool value) {
 
 static void InfoHeader(const char* label) {
     ImGui::NewLine();
-    ImGui::Text(label);
+    ImGui::Text("%s", label);
     ImGui::Separator();
 }
 
@@ -103,7 +103,7 @@ void mvLayoutWindow::renderTreeNode(std::shared_ptr<mvAppItem>& item)
     {
         is_labeled = false;
         labelToShow = DearPyGui::GetEntityTypeString(item->type);
-        constexpr char* prefix = "mvAppItemType::";
+        constexpr const char* prefix = "mvAppItemType::";
         // This is a cumbersome way to get string length, but it's the only way
         // that provides a constexpr (note: strlen() is not a constexpr).
         // Well, we could also do a `sizeof(prefix) - 1`...