Removed some FIXMEs that aren't valid.
authorRyan C. Gordon <icculus@icculus.org>
Sun, 06 Aug 2017 13:56:45 -0400
changeset 1543 210b82b60730
parent 1542 1027fe6accf1
child 1544 7f6020ee746c
Removed some FIXMEs that aren't valid.
src/physfs.c
src/physfs_archiver_zip.c
src/physfs_platform_macos.c
src/physfs_platform_posix.c
src/physfs_platform_unix.c
src/physfs_platform_windows.c
--- a/src/physfs.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs.c	Sun Aug 06 13:56:45 2017 -0400
@@ -2882,7 +2882,7 @@
     stat->createtime = -1;
     stat->accesstime = -1;
     stat->filetype = PHYSFS_FILETYPE_OTHER;
-    stat->readonly = 1;  /* !!! FIXME */
+    stat->readonly = 1;
 
     if (sanitizePlatformIndependentPath(_fname, fname))
     {
@@ -2904,7 +2904,7 @@
                 if (exists)
                 {
                     stat->filetype = PHYSFS_FILETYPE_DIRECTORY;
-                    stat->readonly = 1;  /* !!! FIXME */
+                    stat->readonly = 1;
                     retval = 1;
                 } /* if */
                 else if (verifyPath(i, &arcfname, 0))
--- a/src/physfs_archiver_zip.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs_archiver_zip.c	Sun Aug 06 13:56:45 2017 -0400
@@ -1637,8 +1637,6 @@
     ZIPinfo *info = (ZIPinfo *) opaque;
     const ZIPentry *entry = zip_find_entry(info, filename);
 
-    /* !!! FIXME: does this need to resolve entries here? */
-
     if (entry == NULL)
         return 0;
 
--- a/src/physfs_platform_macos.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs_platform_macos.c	Sun Aug 06 13:56:45 2017 -0400
@@ -1,5 +1,5 @@
 /*
- * Mac OS X support routines for PhysicsFS.
+ * macOS (iOS, etc) support routines for PhysicsFS.
  *
  * Please see the file LICENSE.txt in the source's root directory.
  *
--- a/src/physfs_platform_posix.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs_platform_posix.c	Sun Aug 06 13:56:45 2017 -0400
@@ -331,8 +331,7 @@
     st->createtime = statbuf.st_ctime;
     st->accesstime = statbuf.st_atime;
 
-    /* !!! FIXME: maybe we should just report full permissions? */
-    st->readonly = access(filename, W_OK);
+    st->readonly = (access(filename, W_OK) == -1);
     return 1;
 } /* __PHYSFS_platformStat */
 
--- a/src/physfs_platform_unix.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs_platform_unix.c	Sun Aug 06 13:56:45 2017 -0400
@@ -118,8 +118,6 @@
         else if (strcmp(ent->mnt_type, "supermount") == 0)
             add_it = 1;
 
-        /* !!! FIXME: udf? automount? */
-
         /* add other mount types here */
 
         if (add_it)
--- a/src/physfs_platform_windows.c	Sun Aug 06 12:27:18 2017 -0400
+++ b/src/physfs_platform_windows.c	Sun Aug 06 13:56:45 2017 -0400
@@ -196,7 +196,6 @@
         case ERROR_WRITE_FAULT: return PHYSFS_ERR_IO;
         case ERROR_READ_FAULT: return PHYSFS_ERR_IO;
         case ERROR_DEV_NOT_EXIST: return PHYSFS_ERR_IO;
-        /* !!! FIXME: ?? case ELOOP: return PHYSFS_ERR_SYMLINK_LOOP; */
         case ERROR_BUFFER_OVERFLOW: return PHYSFS_ERR_BAD_FILENAME;
         case ERROR_INVALID_NAME: return PHYSFS_ERR_BAD_FILENAME;
         case ERROR_BAD_PATHNAME: return PHYSFS_ERR_BAD_FILENAME;
@@ -207,8 +206,6 @@
         case ERROR_INVALID_DRIVE: return PHYSFS_ERR_NOT_FOUND;
         case ERROR_HANDLE_DISK_FULL: return PHYSFS_ERR_NO_SPACE;
         case ERROR_DISK_FULL: return PHYSFS_ERR_NO_SPACE;
-        /* !!! FIXME: ?? case ENOTDIR: return PHYSFS_ERR_NOT_FOUND; */
-        /* !!! FIXME: ?? case EISDIR: return PHYSFS_ERR_NOT_A_FILE; */
         case ERROR_WRITE_PROTECT: return PHYSFS_ERR_READ_ONLY;
         case ERROR_LOCK_VIOLATION: return PHYSFS_ERR_BUSY;
         case ERROR_SHARING_VIOLATION: return PHYSFS_ERR_BUSY;