From 4899f1871b1c2203b64251c85db626998b662db0 Mon Sep 17 00:00:00 2001
From: Ryan Fleury <ryan.j.fleury@gmail.com>
Date: Wed, 27 Mar 2024 11:19:24 -0700
Subject: [PATCH] correctly close module file handles on module unloading in
 new demon layer

---
 src/dbgi/dbgi.c                    | 3 +--
 src/demon/win32/demon_core_win32.c | 4 ++++
 src/os/core/win32/os_core_win32.c  | 9 ++++++---
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/dbgi/dbgi.c b/src/dbgi/dbgi.c
index 770c5da9..36879117 100644
--- a/src/dbgi/dbgi.c
+++ b/src/dbgi/dbgi.c
@@ -365,8 +365,7 @@ dbgi_binary_close(String8 exe_path)
     if(need_deletion) for(;;)
     {
       os_rw_mutex_drop_w(stripe->rw_mutex);
-      for(U64 start_t = os_now_microseconds();
-          os_now_microseconds() <= start_t + 250;);
+      for(U64 start_t = os_now_microseconds(); os_now_microseconds() <= start_t + 250;);
       os_rw_mutex_take_w(stripe->rw_mutex);
       if(binary->refcount == 0 && ins_atomic_u64_eval(&binary->scope_touch_count) == 0)
       {
diff --git a/src/demon/win32/demon_core_win32.c b/src/demon/win32/demon_core_win32.c
index cb9ee117..42035c76 100644
--- a/src/demon/win32/demon_core_win32.c
+++ b/src/demon/win32/demon_core_win32.c
@@ -151,6 +151,10 @@ dmn_w32_entity_release(DMN_W32_Entity *entity)
       // rjf: free entity
       SLLStackPush(dmn_w32_shared->entities_first_free, t->e);
       t->e->gen += 1;
+      if(t->e->kind == DMN_W32_EntityKind_Module)
+      {
+        CloseHandle(t->e->handle);
+      }
       
       // rjf: remove from id -> entity map
       if(t->e->id != 0)
diff --git a/src/os/core/win32/os_core_win32.c b/src/os/core/win32/os_core_win32.c
index 9ee8c18f..077dce21 100644
--- a/src/os/core/win32/os_core_win32.c
+++ b/src/os/core/win32/os_core_win32.c
@@ -678,7 +678,8 @@ os_file_close(OS_Handle file)
 {
   if(os_handle_match(file, os_handle_zero())) { return; }
   HANDLE handle = (HANDLE)file.u64[0];
-  CloseHandle(handle);
+  BOOL result = CloseHandle(handle);
+  (void)result;
 }
 
 internal U64
@@ -896,7 +897,8 @@ internal void
 os_file_map_close(OS_Handle map)
 {
   HANDLE handle = (HANDLE)map.u64[0];
-  CloseHandle(handle);
+  BOOL result = CloseHandle(handle);
+  (void)result;
 }
 
 internal void *
@@ -939,7 +941,8 @@ os_file_map_view_open(OS_Handle map, OS_AccessFlags flags, Rng1U64 range)
 internal void
 os_file_map_view_close(OS_Handle map, void *ptr)
 {
-  UnmapViewOfFile(ptr);
+  BOOL result = UnmapViewOfFile(ptr);
+  (void)result;
 }
 
 //- rjf: directory iteration