archivers/zip.c
changeset 163 fc2b8ee5b420
parent 161 4fcc863fdc5f
child 194 09c353d30cd4
--- a/archivers/zip.c	Wed Apr 03 07:38:10 2002 +0000
+++ b/archivers/zip.c	Wed Apr 03 07:40:27 2002 +0000
@@ -168,8 +168,6 @@
 
     while (offset > 0)
     {
-        /* !!! - RYAN, CHECK THIS CAST */
-        /* !!! This should be okay since offset will be <= bufsize */
         PHYSFS_uint32 chunk = (offset > bufsize) ? bufsize : (PHYSFS_uint32)offset;
         PHYSFS_sint32 rc = unzReadCurrentFile(fh, buf, chunk);
         BAIL_IF_MACRO(rc == 0, ERR_IO_ERROR, 0);  /* shouldn't happen. */
@@ -273,9 +271,7 @@
 static int version_does_symlinks(uLong version)
 {
     int retval = 0;
-    /* !!! - RYAN, CHECK THIS CAST */
-    /* !!! - You AND the result with 0xFF, so it can't be larger than 0xFF */
-    unsigned char hosttype = (unsigned char)((version >> 8) & 0xFF);
+    PHYSFS_uint8 hosttype = (PHYSFS_uint8) ((version >> 8) & 0xFF);
 
     /*
      * These are the platforms that can build an archive with symlinks,