From 46125c95b38c3be2cf80010922585a553b9d4b19 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <icculus@icculus.org>
Date: Thu, 22 Mar 2012 02:57:29 -0400
Subject: [PATCH] Removed a FIXME.

---
 src/physfs.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/physfs.c b/src/physfs.c
index d87574d9..923b9820 100644
--- a/src/physfs.c
+++ b/src/physfs.c
@@ -2229,10 +2229,7 @@ PHYSFS_File *PHYSFS_openRead(const char *_fname)
 
         GOTO_IF_MACRO(!searchPath, PHYSFS_ERR_NO_SUCH_PATH, openReadEnd);
 
-        /* !!! FIXME: Why aren't we using a for loop here? */
-        i = searchPath;
-
-        do
+        for (i = searchPath; (i != NULL) && (!fileExists); i = i->next)
         {
             char *arcfname = fname;
             if (verifyPath(i, &arcfname, 0))
@@ -2241,8 +2238,7 @@ PHYSFS_File *PHYSFS_openRead(const char *_fname)
                 if (io)
                     break;
             } /* if */
-            i = i->next;
-        } while ((i != NULL) && (!fileExists));
+        } /* for */
 
         /* !!! FIXME: may not set an error if openRead didn't fail. */
         GOTO_IF_MACRO(!io, ERRPASS, openReadEnd);