[PATCH 02/11] syscalls.cc: Deduplicate _remove_r
Ben
ben@wijen.net
Mon Jan 18 13:51:52 GMT 2021
On 18-01-2021 14:04, Corinna Vinschen via Cygwin-patches wrote:
> What about this instead? It should be better optimizable:
>
Hmmm:
* _remove_r should still set reent->_errno
* _GLOBAL_REENT isn't threadlocal, what about __getreent()
So maybe:
diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc
index 4742c6653..9c498cd46 100644
--- a/winsup/cygwin/syscalls.cc
+++ b/winsup/cygwin/syscalls.cc
@@ -1122,35 +1122,28 @@ unlink (const char *ourname)
}
extern "C" int
-_remove_r (struct _reent *, const char *ourname)
+_remove_r (struct _reent *ptr, const char *ourname)
{
path_conv win32_name (ourname, PC_SYM_NOFOLLOW);
if (win32_name.error)
{
- set_errno (win32_name.error);
+ ptr->_errno = set_errno (win32_name.error);
syscall_printf ("%R = remove(%s)",-1, ourname);
return -1;
}
- return win32_name.isdir () ? rmdir (ourname) : unlink (ourname);
+ int res = win32_name.isdir () ? rmdir (ourname) : unlink (ourname);
+ if (errno!=0)
+ ptr->_errno = errno;
+ syscall_printf ("%R = remove(%s)", res, ourname);
+ return res;
}
extern "C" int
remove (const char *ourname)
{
- path_conv win32_name (ourname, PC_SYM_NOFOLLOW);
-
- if (win32_name.error)
- {
- set_errno (win32_name.error);
- syscall_printf ("-1 = remove (%s)", ourname);
- return -1;
- }
-
- int res = win32_name.isdir () ? rmdir (ourname) : unlink (ourname);
- syscall_printf ("%R = remove(%s)", res, ourname);
- return res;
+ return _remove_r (__getreent (), ourname);
}
extern "C" pid_t
Ben...
More information about the Cygwin-patches
mailing list