Skip to content
Permalink
Browse files
Pushed a bunch of FIXMEs into the Windows platform layer.
  • Loading branch information
icculus committed Mar 25, 2007
1 parent 4fe2134 commit 75a4f4ffe17bf0d18a2f66daafd7e562bb944d40
Showing with 47 additions and 9 deletions.
  1. +47 −9 platform/windows.c
@@ -26,12 +26,15 @@
#define HIGHORDER_UINT64(pos) (PHYSFS_uint32) \
(((pos & 0xFFFFFFFF00000000) >> 32) & 0x00000000FFFFFFFF)


/* !!! FIXME: unicode version. */
/* GetUserProfileDirectory() is only available on >= NT4 (no 9x/ME systems!) */
typedef BOOL (STDMETHODCALLTYPE FAR * LPFNGETUSERPROFILEDIR) (
HANDLE hToken,
LPTSTR lpProfileDir,
LPDWORD lpcchSize);

/* !!! FIXME: unicode version. */
/* GetFileAttributesEx() is only available on >= Win98 or WinNT4 ... */
typedef BOOL (STDMETHODCALLTYPE FAR * LPFNGETFILEATTRIBUTESEX) (
LPCTSTR lpFileName,
@@ -73,6 +76,7 @@ static const char *win32strerror(void)
static TCHAR msgbuf[255];
TCHAR *ptr = msgbuf;

/* !!! FIXME: unicode version. */
FormatMessage(
FORMAT_MESSAGE_FROM_SYSTEM |
FORMAT_MESSAGE_IGNORE_INSERTS,
@@ -84,7 +88,7 @@ static const char *win32strerror(void)
NULL
);

/* chop off newlines. */
/* chop off newlines. */
for (ptr = msgbuf; *ptr; ptr++)
{
if ((*ptr == '\n') || (*ptr == '\r'))
@@ -94,6 +98,8 @@ static const char *win32strerror(void)
} /* if */
} /* for */

/* !!! FIXME: convert to UTF-8. */

return((const char *) msgbuf);
} /* win32strerror */

@@ -110,6 +116,7 @@ static char *getExePath(const char *argv0)
retval[0] = '\0';
/* !!! FIXME: don't preallocate here? */
/* !!! FIXME: use smallAlloc? */
/* !!! FIXME: unicode version. */
buflen = GetModuleFileName(NULL, retval, MAX_PATH + 1);
if (buflen <= 0)
__PHYSFS_setError(win32strerror());
@@ -137,10 +144,11 @@ static char *getExePath(const char *argv0)

if (!success)
{
if (argv0 == NULL)
if (argv0 == NULL) /* !!! FIXME: default behaviour does this. */
__PHYSFS_setError(ERR_ARGV0_IS_NULL);
else
{
/* !!! FIXME: unicode version. */
buflen = SearchPath(NULL, argv0, NULL, MAX_PATH+1, retval, &ptr);
if (buflen == 0)
__PHYSFS_setError(win32strerror());
@@ -198,7 +206,7 @@ static int determineUserDir(void)
lib = LoadLibrary("userenv.dll");
if (lib)
{
/* !!! FIXME: Handle Unicode? */
/* !!! FIXME: unicode version. */
GetUserProfileDirectory = (LPFNGETUSERPROFILEDIR)
GetProcAddress(lib, "GetUserProfileDirectoryA");
if (GetUserProfileDirectory)
@@ -211,18 +219,24 @@ static int determineUserDir(void)
* psize. Also note that the second parameter can't be
* NULL or the function fails.
*/
/* !!! FIXME: unicode version. */
rc = GetUserProfileDirectory(accessToken, dummy, &psize);
assert(!rc); /* success?! */

/* Allocate memory for the profile directory */
userDir = (char *) allocator.Malloc(psize);
if (userDir != NULL)
{
if (!GetUserProfileDirectory(accessToken, userDir, &psize))
/* !!! FIXME: unicode version. */
if (GetUserProfileDirectory(accessToken, userDir, &psize))
{
/* !!! FIXME: convert to UTF-8. */
} /* if */
else
{
allocator.Free(userDir);
userDir = NULL;
} /* if */
} /* else */
} /* else */
} /* if */

@@ -237,6 +251,7 @@ static int determineUserDir(void)
/* Might just be a non-NT system; resort to the basedir. */
userDir = getExePath(NULL);
BAIL_IF_MACRO(userDir == NULL, NULL, 0); /* STILL failed?! */
/* !!! FIXME: convert to UTF-8. */
} /* if */

return(1); /* We made it: hit the showers. */
@@ -264,6 +279,8 @@ static BOOL mediaInDrive(const char *drive)

void __PHYSFS_platformDetectAvailableCDs(PHYSFS_StringCallback cb, void *data)
{
/* !!! FIXME: Can CD drives be non-drive letter paths? */
/* !!! FIXME: (so can they be Unicode paths?) */
char drive_str[4] = "x:\\";
char ch;
for (ch = 'A'; ch <= 'Z'; ch++)
@@ -289,10 +306,12 @@ char *__PHYSFS_platformGetUserName(void)
DWORD bufsize = 0;
LPTSTR retval = NULL;

/* !!! FIXME: unicode version. */
if (GetUserName(NULL, &bufsize) == 0) /* This SHOULD fail. */
{
retval = (LPTSTR) allocator.Malloc(bufsize);
BAIL_IF_MACRO(retval == NULL, ERR_OUT_OF_MEMORY, NULL);
/* !!! FIXME: unicode version. */
if (GetUserName(retval, &bufsize) == 0) /* ?! */
{
__PHYSFS_setError(win32strerror());
@@ -301,6 +320,11 @@ char *__PHYSFS_platformGetUserName(void)
} /* if */
} /* if */

if (retval != NULL)
{
/* !!! FIXME: convert to UTF-8. */
} /* if */

return((char *) retval);
} /* __PHYSFS_platformGetUserName */

@@ -324,6 +348,7 @@ int __PHYSFS_platformExists(const char *fname)
{
BAIL_IF_MACRO
(
/* !!! FIXME: unicode version. */
GetFileAttributes(fname) == PHYSFS_INVALID_FILE_ATTRIBUTES,
win32strerror(), 0
);
@@ -333,12 +358,14 @@ int __PHYSFS_platformExists(const char *fname)

int __PHYSFS_platformIsSymLink(const char *fname)
{
/* !!! FIXME: Vista has symlinks. Recheck this. */
return(0); /* no symlinks on win32. */
} /* __PHYSFS_platformIsSymlink */


int __PHYSFS_platformIsDirectory(const char *fname)
{
/* !!! FIXME: unicode version. */
return((GetFileAttributes(fname) & FILE_ATTRIBUTE_DIRECTORY) != 0);
} /* __PHYSFS_platformIsDirectory */

@@ -401,16 +428,19 @@ void __PHYSFS_platformEnumerateFiles(const char *dirname,
/* Append the "*" to the end of the string */
strcat(SearchPath, "*");

/* !!! FIXME: unicode version. */
dir = FindFirstFile(SearchPath, &ent);
__PHYSFS_smallFree(SearchPath);
if (dir == INVALID_HANDLE_VALUE)
return;

do
{
/* !!! FIXME: unicode version. */
if (strcmp(ent.cFileName, ".") == 0)
continue;

/* !!! FIXME: unicode version. */
if (strcmp(ent.cFileName, "..") == 0)
continue;

@@ -426,9 +456,11 @@ char *__PHYSFS_platformCurrentDir(void)
LPTSTR retval;
DWORD buflen = 0;

/* !!! FIXME: unicode version. */
buflen = GetCurrentDirectory(buflen, NULL);
retval = (LPTSTR) allocator.Malloc(sizeof (TCHAR) * (buflen + 2));
BAIL_IF_MACRO(retval == NULL, ERR_OUT_OF_MEMORY, NULL);
/* !!! FIXME: unicode version. */
GetCurrentDirectory(buflen, retval);

if (retval[buflen - 2] != '\\')
@@ -441,6 +473,7 @@ char *__PHYSFS_platformCurrentDir(void)
/* this could probably use a cleanup. */
char *__PHYSFS_platformRealPath(const char *path)
{
/* this function should be UTF-8 clean. */
char *retval = NULL;
char *p = NULL;

@@ -516,15 +549,15 @@ char *__PHYSFS_platformRealPath(const char *path)
p = retval;
while ( (p = strstr(p, "\\.")) != NULL)
{
/* it's a "." entry that doesn't end the string. */
/* it's a "." entry that doesn't end the string. */
if (p[2] == '\\')
memmove(p + 1, p + 3, strlen(p + 3) + 1);

/* it's a "." entry that ends the string. */
/* it's a "." entry that ends the string. */
else if (p[2] == '\0')
p[0] = '\0';

/* it's a ".." entry. */
/* it's a ".." entry. */
else if (p[2] == '.')
{
char *prevEntry = p - 1;
@@ -550,7 +583,7 @@ char *__PHYSFS_platformRealPath(const char *path)
} /* else */
} /* while */

/* shrink the retval's memory block if possible... */
/* shrink the retval's memory block if possible... */
p = (char *) allocator.Realloc(retval, strlen(retval) + 1);
if (p != NULL)
retval = p;
@@ -561,6 +594,7 @@ char *__PHYSFS_platformRealPath(const char *path)

int __PHYSFS_platformMkDir(const char *path)
{
/* !!! FIXME: unicode version. */
DWORD rc = CreateDirectory(path, NULL);
BAIL_IF_MACRO(rc == 0, win32strerror(), 0);
return(1);
@@ -663,6 +697,7 @@ static void *doOpen(const char *fname, DWORD mode, DWORD creation, int rdonly)
HANDLE fileHandle;
win32file *retval;

/* !!! FIXME: unicode version. */
fileHandle = CreateFile(fname, mode, FILE_SHARE_READ, NULL,
creation, FILE_ATTRIBUTE_NORMAL, NULL);

@@ -892,10 +927,12 @@ int __PHYSFS_platformDelete(const char *path)
/* If filename is a folder */
if (GetFileAttributes(path) == FILE_ATTRIBUTE_DIRECTORY)
{
/* !!! FIXME: unicode version. */
BAIL_IF_MACRO(!RemoveDirectory(path), win32strerror(), 0);
} /* if */
else
{
/* !!! FIXME: unicode version. */
BAIL_IF_MACRO(!DeleteFile(path), win32strerror(), 0);
} /* else */

@@ -1019,6 +1056,7 @@ PHYSFS_sint64 __PHYSFS_platformGetLastModTime(const char *fname)
/* GetFileAttributesEx didn't show up until Win98 and NT4. */
if (pGetFileAttributesEx != NULL)
{
/* !!! FIXME: unicode version. */
if (pGetFileAttributesEx(fname, GetFileExInfoStandard, &attrData))
{
/* 0 return value indicates an error or not supported */

0 comments on commit 75a4f4f

Please sign in to comment.