You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2020/03/11 06:14:21 UTC
[incubator-nuttx] branch master updated: include/nuttx/compiler.h:
Appease nxstyle errors
This is an automated email from the ASF dual-hosted git repository.
xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git
The following commit(s) were added to refs/heads/master by this push:
new 14d037b include/nuttx/compiler.h: Appease nxstyle errors
14d037b is described below
commit 14d037b8d50a6fa63fbcb249df7aa3fb79a8ce6b
Author: YAMAMOTO Takashi <ya...@midokura.com>
AuthorDate: Wed Mar 11 13:50:55 2020 +0900
include/nuttx/compiler.h: Appease nxstyle errors
---
include/nuttx/compiler.h | 32 ++++++++++++++++++--------------
1 file changed, 18 insertions(+), 14 deletions(-)
diff --git a/include/nuttx/compiler.h b/include/nuttx/compiler.h
index 28e41ad..bf9d222 100644
--- a/include/nuttx/compiler.h
+++ b/include/nuttx/compiler.h
@@ -72,7 +72,8 @@
/* GCC 4.x have __builtin_ctz(|l|ll) and __builtin_clz(|l|ll). These count
* trailing/leading zeros of input number and typically will generate few
* fast bit-counting instructions. Inputting zero to these functions is
- * undefined and needs to be taken care of by the caller. */
+ * undefined and needs to be taken care of by the caller.
+ */
#if __GNUC__ >= 4
# define CONFIG_HAVE_BUILTIN_CTZ 1
@@ -140,8 +141,8 @@
# define naked_function __attribute__ ((naked,no_instrument_function))
/* The inline_function attribute informs GCC that the function should always
- * be inlined, regardless of the level of optimization. The noinline_function
- * indicates that the function should never be inlined.
+ * be inlined, regardless of the level of optimization. The
+ * noinline_function indicates that the function should never be inlined.
*/
# define inline_function __attribute__ ((always_inline,no_instrument_function))
@@ -277,8 +278,8 @@
* may support them).
*
* CAREFUL: This can cause issues for shared data structures shared between
- * C and C++ if the two versions do not support the same features. Structures
- * and unions can lose binary compatibility!
+ * C and C++ if the two versions do not support the same features.
+ * Structures and unions can lose binary compatibility!
*
* NOTE: The NuttX coding standard forbids the use of unnamed structures and
* unions within the OS.
@@ -328,7 +329,8 @@
/* Pragmas
*
- * Disable warnings for unused function arguments */
+ * Disable warnings for unused function arguments
+ */
# pragma disable_warning 85
@@ -338,7 +340,8 @@
/* Attributes
*
- * SDCC does not support weak symbols */
+ * SDCC does not support weak symbols
+ */
# undef CONFIG_HAVE_WEAKFUNCTIONS
# define weak_alias(name, aliasname)
@@ -347,6 +350,7 @@
# define restrict /* REVISIT */
/* SDCC does not support the noreturn or packed attributes */
+
/* Current SDCC supports noreturn via C11 _Noreturn keyword (see
* stdnoreturn.h).
*/
@@ -519,8 +523,8 @@
* Z8Encore!: Far is 16-bits; near is 8-bits of address.
* The supported model is (1) all code on ROM, and (2) all data
* and stacks in internal (far) RAM.
- * Z8Acclaim: In Z80 mode, all pointers are 16-bits. In ADL mode, all pointers
- * are 24 bits.
+ * Z8Acclaim: In Z80 mode, all pointers are 16-bits. In ADL mode, all
+ * pointers are 24 bits.
*/
# if defined(__ZNEO__)
@@ -565,9 +569,9 @@
# undef CONFIG_HAVE_ANONYMOUS_STRUCT
# undef CONFIG_HAVE_ANONYMOUS_UNION
-/* Older Zilog compilers support both types double and long long, but the size
- * is 32-bits (same as long and single precision) so it is safer to say that
- * they are not supported. Later versions are more ANSII compliant and
+/* Older Zilog compilers support both types double and long long, but the
+ * size is 32-bits (same as long and single precision) so it is safer to say
+ * that they are not supported. Later versions are more ANSII compliant and
* simply do not support long long or double.
*/
@@ -584,12 +588,12 @@
# define UNUSED(a) ((void)(a))
-/* ICCARM-specific definitions ***********************************************/
+/* ICCARM-specific definitions **********************************************/
#elif defined(__ICCARM__)
# define CONFIG_CPP_HAVE_VARARGS 1 /* Supports variable argument macros */
-# define CONFIG_HAVE_FILENAME 1 /* Has __FILE__ */
+# define CONFIG_HAVE_FILENAME 1 /* Has __FILE__ */
# define CONFIG_HAVE_FLOAT 1
/* Indicate that a local variable is not used */