summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2017-08-16 20:39:18 -0500
committerEric Blake <eblake@redhat.com>2017-08-17 07:10:03 -0500
commit979d467ff6e39ee5c52cf1aac7a6c9c63058141c (patch)
tree0fc9ef1dde22b6a84cc142bbe17465e94df21953
parent7b3d8b9485186aa17db10c76f347e3e12201a12c (diff)
downloadcygnal-979d467ff6e39ee5c52cf1aac7a6c9c63058141c.tar.gz
cygnal-979d467ff6e39ee5c52cf1aac7a6c9c63058141c.tar.bz2
cygnal-979d467ff6e39ee5c52cf1aac7a6c9c63058141c.zip
headers: avoid bareword attributes
Always use the __-decorated form of an attribute name in public headers, as the bareword form is in the user's namespace, and we don't want compilation to break just because the user defines the bareword to mean something else. Signed-off-by: Eric Blake <eblake@redhat.com>
-rw-r--r--newlib/libc/include/sys/cdefs.h28
-rw-r--r--newlib/libc/sys/phoenix/sys/unistd.h2
2 files changed, 15 insertions, 15 deletions
diff --git a/newlib/libc/include/sys/cdefs.h b/newlib/libc/include/sys/cdefs.h
index 2e63a07a2..9e58ee980 100644
--- a/newlib/libc/include/sys/cdefs.h
+++ b/newlib/libc/include/sys/cdefs.h
@@ -681,42 +681,42 @@
#endif
/* Structure implements a lock. */
-#define __lockable __lock_annotate(lockable)
+#define __lockable __lock_annotate(__lockable__)
/* Function acquires an exclusive or shared lock. */
#define __locks_exclusive(...) \
- __lock_annotate(exclusive_lock_function(__VA_ARGS__))
+ __lock_annotate(__exclusive_lock_function__(__VA_ARGS__))
#define __locks_shared(...) \
- __lock_annotate(shared_lock_function(__VA_ARGS__))
+ __lock_annotate(__shared_lock_function__(__VA_ARGS__))
/* Function attempts to acquire an exclusive or shared lock. */
#define __trylocks_exclusive(...) \
- __lock_annotate(exclusive_trylock_function(__VA_ARGS__))
+ __lock_annotate(__exclusive_trylock_function__(__VA_ARGS__))
#define __trylocks_shared(...) \
- __lock_annotate(shared_trylock_function(__VA_ARGS__))
+ __lock_annotate(__shared_trylock_function__(__VA_ARGS__))
/* Function releases a lock. */
-#define __unlocks(...) __lock_annotate(unlock_function(__VA_ARGS__))
+#define __unlocks(...) __lock_annotate(__unlock_function__(__VA_ARGS__))
/* Function asserts that an exclusive or shared lock is held. */
#define __asserts_exclusive(...) \
- __lock_annotate(assert_exclusive_lock(__VA_ARGS__))
+ __lock_annotate(__assert_exclusive_lock__(__VA_ARGS__))
#define __asserts_shared(...) \
- __lock_annotate(assert_shared_lock(__VA_ARGS__))
+ __lock_annotate(__assert_shared_lock__(__VA_ARGS__))
/* Function requires that an exclusive or shared lock is or is not held. */
#define __requires_exclusive(...) \
- __lock_annotate(exclusive_locks_required(__VA_ARGS__))
+ __lock_annotate(__exclusive_locks_required__(__VA_ARGS__))
#define __requires_shared(...) \
- __lock_annotate(shared_locks_required(__VA_ARGS__))
+ __lock_annotate(__shared_locks_required__(__VA_ARGS__))
#define __requires_unlocked(...) \
- __lock_annotate(locks_excluded(__VA_ARGS__))
+ __lock_annotate(__locks_excluded__(__VA_ARGS__))
/* Function should not be analyzed. */
-#define __no_lock_analysis __lock_annotate(no_thread_safety_analysis)
+#define __no_lock_analysis __lock_annotate(__no_thread_safety_analysis__)
/* Guard variables and structure members by lock. */
-#define __guarded_by(x) __lock_annotate(guarded_by(x))
-#define __pt_guarded_by(x) __lock_annotate(pt_guarded_by(x))
+#define __guarded_by(x) __lock_annotate(__guarded_by__(x))
+#define __pt_guarded_by(x) __lock_annotate(__pt_guarded_by__(x))
#endif /* !_SYS_CDEFS_H_ */
diff --git a/newlib/libc/sys/phoenix/sys/unistd.h b/newlib/libc/sys/phoenix/sys/unistd.h
index bdf5bd529..77b96ddd4 100644
--- a/newlib/libc/sys/phoenix/sys/unistd.h
+++ b/newlib/libc/sys/phoenix/sys/unistd.h
@@ -35,7 +35,7 @@
extern char **environ;
-void _exit(int status) _ATTRIBUTE ((noreturn));
+void _exit(int status) _ATTRIBUTE ((__noreturn__));
int access(const char *pathname, int mode);
unsigned int alarm(unsigned seconds);