r940 - in trunk: . platform
DONOTREPLY at icculus.org
DONOTREPLY at icculus.org
Thu Apr 3 01:05:48 EDT 2008
Author: icculus
Date: 2008-04-03 01:05:48 -0400 (Thu, 03 Apr 2008)
New Revision: 940
Modified:
trunk/CHANGELOG.txt
trunk/platform/posix.c
Log:
Fixed PHYSFS_openAppend() on Unix.
Apparently O_APPEND doesn't behave like I thought it did...all these years. :/
Modified: trunk/CHANGELOG.txt
===================================================================
--- trunk/CHANGELOG.txt 2008-03-12 21:42:16 UTC (rev 939)
+++ trunk/CHANGELOG.txt 2008-04-03 05:05:48 UTC (rev 940)
@@ -2,6 +2,7 @@
* CHANGELOG.
*/
+04032008 - Fixed PHYSFS_openAppend() to work as documented on Unix.
03122008 - Fixed aliasing bug in Windows platform layer (thanks, Dennis!).
03102008 - Changed some text files from ISO-8859-1 to UTF-8. Replaced all the
translations with UTF-8 encoded equivalents.
Modified: trunk/platform/posix.c
===================================================================
--- trunk/platform/posix.c 2008-03-12 21:42:16 UTC (rev 939)
+++ trunk/platform/posix.c 2008-04-03 05:05:48 UTC (rev 940)
@@ -236,13 +236,26 @@
static void *doOpen(const char *filename, int mode)
{
+ const int appending = (mode & O_APPEND);
int fd;
int *retval;
errno = 0;
+ /* O_APPEND doesn't actually behave as we'd like. */
+ mode &= ~O_APPEND;
+
fd = open(filename, mode, S_IRUSR | S_IWUSR);
BAIL_IF_MACRO(fd < 0, strerror(errno), NULL);
+ if (appending)
+ {
+ if (lseek(fd, 0, SEEK_END) < 0)
+ {
+ close(fd);
+ BAIL_MACRO(strerror(errno), NULL);
+ } /* if */
+ } /* if */
+
retval = (int *) allocator.Malloc(sizeof (int));
if (retval == NULL)
{
More information about the physfs-commits
mailing list