From 6099f76c8c9f67c815ba147506451008616d9282 Mon Sep 17 00:00:00 2001
From: Daan Leijen <daan@microsoft.com>
Date: Fri, 3 Jan 2025 14:26:32 -0800
Subject: [PATCH] nicer logic in free

---
 src/free.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/free.c b/src/free.c
index f63a55cb..5efe0280 100644
--- a/src/free.c
+++ b/src/free.c
@@ -176,18 +176,16 @@ void mi_free(void* p) mi_attr_noexcept
     // page is local, but is full or contains (inner) aligned blocks; use generic path
     mi_free_generic_local(page, p);
   }
-  else {
-      // free-ing in a page owned by a heap in another thread, or on abandoned page (not belonging to a heap)
-    if ((xtid & MI_PAGE_FLAG_MASK) == 0) {         // `tid!=mi_page_thread_id(page) && mi_page_flags(page)==0`
-      // blocks are aligned (and not a full page)
-      mi_block_t* const block = (mi_block_t*)p;
-      mi_free_block_mt(page,block);
-    }
-    else {
-      // page is full or contains (inner) aligned blocks; use generic multi-thread path
-      mi_free_generic_mt(page, p);
-    }
+  // free-ing in a page owned by a heap in another thread, or on abandoned page (not belonging to a heap)
+  else if ((xtid & MI_PAGE_FLAG_MASK) == 0) {         // `tid!=mi_page_thread_id(page) && mi_page_flags(page)==0`
+    // blocks are aligned (and not a full page)
+    mi_block_t* const block = (mi_block_t*)p;
+    mi_free_block_mt(page,block);
   }
+  else {
+    // page is full or contains (inner) aligned blocks; use generic multi-thread path
+    mi_free_generic_mt(page, p);
+  }  
 }