From Lorenz Meier: The implementation of access() as vararg macro has the issue that any function call with the same name (even in a C++ class) will match with it and result in a compile error. I have replaced it with a small function, and tried to have decent documentation as well. This resolves the compile issue, and shouldn’t have negative side effects for users of the function.

1 files changed, 2 insertions, 3 deletions

diff --git a/nuttx/include/unistd.h b/nuttx/include/unistd.h
index 5108faa51..3fbf5698d 100644
--- a/nuttx/include/unistd.h
+++ b/nuttx/include/unistd.h
@@ -167,8 +167,9 @@ FAR char *getcwd(FAR char *buf, size_t size);
 
 /* File path operations */
 
-int     unlink(FAR const char *pathname);
+int     access(FAR const char *path, int amode);
 int     rmdir(FAR const char *pathname);
+int     unlink(FAR const char *pathname);
 
 /* Execution of programs from files */
 
@@ -190,8 +191,6 @@ FAR char **getoptargp(void); /* Optional argument following option */
 int       *getoptindp(void); /* Index into argv */
 int       *getoptoptp(void); /* unrecognized option character */
 
-#define access(...) (0)
-
 #undef EXTERN
 #if defined(__cplusplus)
 }