From f2ebb00ef9303bdb637b16fe96cbdc3e60b77d6f Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Thu, 18 Dec 2003 19:53:20 +0000 Subject: [PATCH] Says Travis Wells: "Meh. I just realized the wad archiver isn't considering ExMx or MAPxx entries correctly, it treats them as files,when it should consider them directories, and place the next 11 entries in that directory.Right now it's making it impossible to open maps, since they all have the same names, they all overlap. So it's not very useful right now. ... Here's the patch to wad.c, it should now correctly place the map-files into subdirectories." Thanks, Travis! --- archivers/wad.c | 75 +++++++++++++++++++++++++++++++++++++------------ 1 file changed, 57 insertions(+), 18 deletions(-) diff --git a/archivers/wad.c b/archivers/wad.c index 3c7fbe62..6b5f470b 100644 --- a/archivers/wad.c +++ b/archivers/wad.c @@ -58,7 +58,7 @@ typedef struct { - char name[9]; + char name[18]; PHYSFS_uint32 startPos; PHYSFS_uint32 size; } WADentry; @@ -309,6 +309,9 @@ static int wad_load_entries(const char *name, int forWriting, WADinfo *info) PHYSFS_uint32 fileCount; PHYSFS_uint32 directoryOffset; WADentry *entry; + char lastDirectory[9],buffer[9]; + + lastDirectory[8] = 0; /* Make sure lastDirectory stays null-terminated. */ BAIL_IF_MACRO(!wad_open(name, forWriting, &fh, &fileCount,&directoryOffset), NULL, 0); info->entryCount = fileCount; @@ -412,34 +415,47 @@ static LinkedStringList *WAD_enumerateFiles(DirHandle *h, LinkedStringList *retval = NULL, *p = NULL; PHYSFS_uint32 max = info->entryCount; PHYSFS_uint32 i; + char *sep; - /* no directories in WAD files. */ - BAIL_IF_MACRO(*dirname != '\0', ERR_NOT_A_DIR, NULL); - - for (i = 0; i < max; i++, entry++) - retval = __PHYSFS_addToLinkedStringList(retval, &p, entry->name, -1); - + if (dirname[0] == 0) + { + for (i = 0; i < max; i++, entry++) + { + if (strchr(entry->name, '/') == NULL) + { + retval = __PHYSFS_addToLinkedStringList(retval, &p, + entry->name, -1); + } /* if */ + } /* for */ + } /* if */ + else + { + for (i = 0; i < max; i++, entry++) + { + sep = strchr(entry->name, '/'); + if (sep != NULL) + { + if (strncmp(dirname, entry->name, (sep-entry->name)) == 0) + { + retval = __PHYSFS_addToLinkedStringList(retval, &p, + sep + 1, -1); + } /* if */ + } /* if */ + } /* for */ + } /* else */ + return(retval); } /* WAD_enumerateFiles */ static WADentry *wad_find_entry(WADinfo *info, const char *name) { - char *ptr = strchr(name, '.'); WADentry *a = info->entries; PHYSFS_sint32 lo = 0; PHYSFS_sint32 hi = (PHYSFS_sint32) (info->entryCount - 1); PHYSFS_sint32 middle; int rc; - /* - * Rule out filenames to avoid unneeded processing...no dirs, - * big filenames, or extensions > 3 chars. - */ - BAIL_IF_MACRO((ptr) && (strlen(ptr) > 4), ERR_NO_SUCH_FILE, NULL); - BAIL_IF_MACRO(strlen(name) > 12, ERR_NO_SUCH_FILE, NULL); - BAIL_IF_MACRO(strchr(name, '/') != NULL, ERR_NO_SUCH_FILE, NULL); - while (lo <= hi) { middle = lo + ((hi - lo) / 2); @@ -464,8 +480,31 @@ static int WAD_exists(DirHandle *h, const char *name) static int WAD_isDirectory(DirHandle *h, const char *name, int *fileExists) { - *fileExists = WAD_exists(h, name); - return(0); /* never directories in a wad. */ + WADentry *entry = wad_find_entry(((WADinfo *) h->opaque), name); + if (entry != NULL) + { + char *n; + + *fileExists = 1; + + /* Can't be a directory if it's a subdirectory. */ + if (strchr(entry->name, '/') != NULL) + return(0); + + /* Check if it matches "MAP??" or "E?M?" ... */ + n = entry->name; + if ((n[0] == 'E' && n[2] == 'M') || + (n[0] == 'M' && n[1] == 'A' && n[2] == 'P' && n[6] == 0)) + { + return(1); + } /* if */ + return(0); + } /* if */ + else + { + *fileExists = 0; + return(0); + } /* else */ } /* WAD_isDirectory */