Răsfoiți Sursa

__PHYSFS_platformDeinit() should return void.

What are you going to do if this fails, anyhow?
Ryan C. Gordon 8 ani în urmă
părinte
comite
61bdee4d78

+ 1 - 2
src/physfs.c

@@ -1362,8 +1362,7 @@ static int doDeinit(void)
 
     errorLock = stateLock = NULL;
 
-    /* !!! FIXME: what on earth are you supposed to do if this fails? */
-    BAIL_IF_ERRPASS(!__PHYSFS_platformDeinit(), 0);
+    __PHYSFS_platformDeinit();
 
     return 1;
 } /* doDeinit */

+ 1 - 4
src/physfs_internal.h

@@ -431,11 +431,8 @@ int __PHYSFS_platformInit(void);
  * Deinitialize the platform. This is called when PHYSFS_deinit() is called
  *  from the application. You can use this to clean up anything you've
  *  allocated in your platform driver.
- *
- * Return zero if there was a catastrophic failure (which prevents you from
- *  functioning at all), and non-zero otherwise.
  */
-int __PHYSFS_platformDeinit(void);
+void __PHYSFS_platformDeinit(void);
 
 
 /*

+ 2 - 2
src/physfs_platform_haiku.cpp

@@ -32,9 +32,9 @@ int __PHYSFS_platformInit(void)
 } /* __PHYSFS_platformInit */
 
 
-int __PHYSFS_platformDeinit(void)
+void __PHYSFS_platformDeinit(void)
 {
-    return 1;  /* always succeed. */
+    /* no-op */
 } /* __PHYSFS_platformDeinit */
 
 

+ 2 - 2
src/physfs_platform_macos.c

@@ -27,9 +27,9 @@ int __PHYSFS_platformInit(void)
 } /* __PHYSFS_platformInit */
 
 
-int __PHYSFS_platformDeinit(void)
+void __PHYSFS_platformDeinit(void)
 {
-    return 1;  /* always succeed. */
+    /* no-op */
 } /* __PHYSFS_platformDeinit */
 
 

+ 1 - 3
src/physfs_platform_os2.c

@@ -271,7 +271,7 @@ int __PHYSFS_platformInit(void)
 } /* __PHYSFS_platformInit */
 
 
-int __PHYSFS_platformDeinit(void)
+void __PHYSFS_platformDeinit(void)
 {
     if (uconvdll)
     {
@@ -280,8 +280,6 @@ int __PHYSFS_platformDeinit(void)
         DosFreeModule(uconvdll);
         uconvdll = 0;
     } /* if */
-
-    return 1;  /* success. */
 } /* __PHYSFS_platformDeinit */
 
 

+ 2 - 2
src/physfs_platform_unix.c

@@ -65,9 +65,9 @@ int __PHYSFS_platformInit(void)
 } /* __PHYSFS_platformInit */
 
 
-int __PHYSFS_platformDeinit(void)
+void __PHYSFS_platformDeinit(void)
 {
-    return 1;  /* always succeed. */
+    /* no-op */
 } /* __PHYSFS_platformDeinit */
 
 

+ 1 - 2
src/physfs_platform_windows.c

@@ -612,10 +612,9 @@ int __PHYSFS_platformInit(void)
 } /* __PHYSFS_platformInit */
 
 
-int __PHYSFS_platformDeinit(void)
+void __PHYSFS_platformDeinit(void)
 {
     deinitCDThread();
-    return 1; /* It's all good */
 } /* __PHYSFS_platformDeinit */