commit e61b792f977cc39e852fa77bc610dd178a1f8d6e
author Alexandre Julliard <julliard@winehq.org> Sat Jul 31 19:24:11 1999 +0000
committer Alexandre Julliard <julliard@winehq.org> Sat Jul 31 19:24:11 1999 +0000
tree 25fcb94557696d310dfed68fd03c0082a5947536
parent 4a0af4223e77d0fc969f0cf1a53c0d4685267570

Remove file from cache when flushing it (based on a patch by Ian
Schmidt).

files/profile.c

diff --git a/files/profile.c b/files/profile.c
index 7b0572e..25b9412 100644
--- a/files/profile.c
+++ b/files/profile.c
@@ -406,6 +406,27 @@
 
 
 /***********************************************************************
+ *           PROFILE_ReleaseFile
+ *
+ * Flush the current profile to disk and remove it from the cache.
+ */
+static void PROFILE_ReleaseFile(void)
+{
+    PROFILE_FlushFile();
+    PROFILE_Free( CurProfile->section );
+    if (CurProfile->dos_name) HeapFree( SystemHeap, 0, CurProfile->dos_name );
+    if (CurProfile->unix_name) HeapFree( SystemHeap, 0, CurProfile->unix_name );
+    if (CurProfile->filename) HeapFree( SystemHeap, 0, CurProfile->filename );
+    CurProfile->changed   = FALSE;
+    CurProfile->section   = NULL;
+    CurProfile->dos_name  = NULL;
+    CurProfile->unix_name = NULL;
+    CurProfile->filename  = NULL;
+    CurProfile->mtime     = 0;
+}
+
+
+/***********************************************************************
  *           PROFILE_Open
  *
  * Open a profile file, checking the cached file first.
@@ -484,20 +505,7 @@
 
     /* Flush the profile */
 
-    if(CurProfile->filename)
-      {
-       PROFILE_FlushFile();
-       PROFILE_Free( CurProfile->section );
-       if (CurProfile->dos_name) HeapFree( SystemHeap, 0, CurProfile->dos_name );
-       if (CurProfile->unix_name) HeapFree( SystemHeap, 0, CurProfile->unix_name );
-       if (CurProfile->filename) HeapFree( SystemHeap, 0, CurProfile->filename );
-       CurProfile->changed=FALSE;
-       CurProfile->section=NULL;
-       CurProfile->dos_name=NULL;
-       CurProfile->unix_name=NULL;
-       CurProfile->filename=NULL;
-       CurProfile->mtime=0;
-      }
+    if(CurProfile->filename) PROFILE_ReleaseFile();
 
     newdos_name = HEAP_strdupA( SystemHeap, 0, full_name.short_name );
     CurProfile->dos_name  = newdos_name;
@@ -1261,20 +1269,19 @@
 BOOL WINAPI WritePrivateProfileStringA( LPCSTR section, LPCSTR entry,
 					LPCSTR string, LPCSTR filename )
 {
-    BOOL	ret;
+    BOOL ret = FALSE;
 
     EnterCriticalSection( &PROFILE_CritSect );
 
-    if (!PROFILE_Open( filename )) {
-       ret = FALSE;
-    } else if (!section) {
-       ret = PROFILE_FlushFile();
-    } else {
-       ret = PROFILE_SetString( section, entry, string );
+    if (PROFILE_Open( filename ))
+    {
+        if (!section && !entry && !string)
+            PROFILE_ReleaseFile();  /* always return FALSE in this case */
+        else
+            ret = PROFILE_SetString( section, entry, string );
     }
 
     LeaveCriticalSection( &PROFILE_CritSect );
-
     return ret;
 }
 
@@ -1497,21 +1504,17 @@
  *           WritePrivateProfileStruct32A (KERNEL32.744)
  */
 BOOL WINAPI WritePrivateProfileStructA (LPCSTR section, LPCSTR key, 
-	LPVOID buf, UINT bufsize, LPCSTR filename)
+                                        LPVOID buf, UINT bufsize, LPCSTR filename)
 {
-    BOOL ret;
+    BOOL ret = FALSE;
+
+    if (!section && !key && !buf)  /* flush the cache */
+        return WritePrivateProfileStringA( NULL, NULL, NULL, filename );
 
     EnterCriticalSection( &PROFILE_CritSect );
 
-    if ((!section) && (!key) && (!buf)) {	/* flush the cache */
-        PROFILE_FlushFile();
-	ret = FALSE;
-    } else {
-       if (!PROFILE_Open( filename )) 
-	  ret = FALSE;
-       else 
-	  ret = PROFILE_SetString( section, key, buf);
-    }
+    if (PROFILE_Open( filename )) 
+        ret = PROFILE_SetString( section, key, buf );
 
     LeaveCriticalSection( &PROFILE_CritSect );