wim.c: replace finalize_wim_struct() with wim_decrement_refcnt()

diff --git a/include/wimlib/wim.h b/include/wimlib/wim.h
index 00de8a53118a74d634b1d3385bfe16f3593015a2..67f55e3bb0ed24df599161cf7fd314ef01b80028 100644 (file)
--- a/include/wimlib/wim.h
+++ b/include/wimlib/wim.h
@@ -186,7 +186,7 @@ static inline bool wim_is_pipable(const WIMStruct *wim)
 }
 
 extern void
-finalize_wim_struct(WIMStruct *wim);
+wim_decrement_refcnt(WIMStruct *wim);
 
 extern bool
 wim_has_solid_resources(const WIMStruct *wim);

diff --git a/src/blob_table.c b/src/blob_table.c
index 1509a835489dcb8daed0389e82b4b193bc5d13e9..e1fb6adac77c3c46603468c9e1be058d450270f7 100644 (file)
--- a/src/blob_table.c
+++ b/src/blob_table.c
@@ -173,9 +173,7 @@ blob_release_location(struct blob_descriptor *blob)
 
                list_del(&blob->rdesc_node);
                if (list_empty(&rdesc->blob_list)) {
-                       wimlib_assert(rdesc->wim->refcnt > 0);
-                       if (--rdesc->wim->refcnt == 0)
-                               finalize_wim_struct(rdesc->wim);
+                       wim_decrement_refcnt(rdesc->wim);
                        FREE(rdesc);
                }
                break;

diff --git a/src/wim.c b/src/wim.c
index e01131ddffabfdc1504fc3b828d762f880ba526e..b5827171ba91e139d9f7a5c83cc756d8b08041c0 100644 (file)
--- a/src/wim.c
+++ b/src/wim.c
@@ -867,10 +867,14 @@ can_modify_wim(WIMStruct *wim)
        return 0;
 }
 
-/* Free a WIMStruct after no more resources reference it.  */
+/* Release a reference to a WIMStruct.  If the reference count reaches 0, the
+ * WIMStruct is freed.  */
 void
-finalize_wim_struct(WIMStruct *wim)
+wim_decrement_refcnt(WIMStruct *wim)
 {
+       wimlib_assert(wim->refcnt > 0);
+       if (--wim->refcnt != 0)
+               return;
        if (filedes_valid(&wim->in_fd))
                filedes_close(&wim->in_fd);
        if (filedes_valid(&wim->out_fd))
@@ -901,9 +905,7 @@ wimlib_free(WIMStruct *wim)
                wim->image_metadata = NULL;
        }
 
-       wimlib_assert(wim->refcnt > 0);
-       if (--wim->refcnt == 0)
-               finalize_wim_struct(wim);
+       wim_decrement_refcnt(wim);
 }
 
 static bool