From: Tor Lillqvist <tml@novell.com>
Date: Fri, 31 Mar 2006 19:16:23 +0000 (+0000)
Subject: Revert change below. It wasn't a good idea after all, says the original
X-Git-Tag: GLIB_2_11_0~74
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b2438cad5148539e4bc8764a8a2d638a87510e7d;p=platform%2Fupstream%2Fglib.git

Revert change below. It wasn't a good idea after all, says the original

2006-03-31  Tor Lillqvist  <tml@novell.com>

	* glib/gstdio.c (g_remove): Revert change below. It wasn't a good
	idea after all, says the original bug reporter. See bug for
	discussion.
---

diff --git a/ChangeLog b/ChangeLog
index a537837..95f90c8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-03-31  Tor Lillqvist  <tml@novell.com>
+
+	* glib/gstdio.c (g_remove): Revert change below. It wasn't a good
+	idea after all, says the original bug reporter. See bug for
+	discussion.
+
 2006-03-30  Tor Lillqvist  <tml@novell.com>
 
 	* glib/gstdio.c (g_remove): [Win32] call rmdir() only if remove()
diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12
index a537837..95f90c8 100644
--- a/ChangeLog.pre-2-12
+++ b/ChangeLog.pre-2-12
@@ -1,3 +1,9 @@
+2006-03-31  Tor Lillqvist  <tml@novell.com>
+
+	* glib/gstdio.c (g_remove): Revert change below. It wasn't a good
+	idea after all, says the original bug reporter. See bug for
+	discussion.
+
 2006-03-30  Tor Lillqvist  <tml@novell.com>
 
 	* glib/gstdio.c (g_remove): [Win32] call rmdir() only if remove()
diff --git a/glib/gstdio.c b/glib/gstdio.c
index ac13ef6..b772591 100644
--- a/glib/gstdio.c
+++ b/glib/gstdio.c
@@ -758,6 +758,11 @@ g_unlink (const gchar *filename)
  * Windows, it is in general not possible to remove a file that is
  * open to some process, or mapped into memory.
  *
+ * If this function fails on Windows you can't infer too much from the
+ * errno value. rmdir() is tried regardless of what caused remove() to
+ * fail. Any errno value set by remove() will be overwritten by that
+ * set by rmdir().
+ *
  * Returns: 0 if the file was successfully removed, -1 if an error 
  *    occurred
  * 
@@ -780,7 +785,7 @@ g_remove (const gchar *filename)
 	}
 
       retval = _wremove (wfilename);
-      if (retval == -1 && errno == ENOENT)
+      if (retval == -1)
 	retval = _wrmdir (wfilename);
       save_errno = errno;
 
@@ -802,7 +807,7 @@ g_remove (const gchar *filename)
 	}
 
       retval = remove (cp_filename);
-      if (retval == -1 && errno == ENOENT)
+      if (retval == -1)
 	retval = rmdir (cp_filename);
       save_errno = errno;