diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2015-02-19 11:47:34 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2015-02-19 11:47:34 -0600 |
commit | 07b5b43b323334b02fab26754dc6e0cdd3154631 (patch) | |
tree | d0f7bc8ca88071ddc2a6f8b58eb4cf4350f66d51 /nuttx/include | |
parent | def91b011ffcbc84cf743c87d36e5e9fb9e26b54 (diff) | |
download | px4-nuttx-07b5b43b323334b02fab26754dc6e0cdd3154631.tar.gz px4-nuttx-07b5b43b323334b02fab26754dc6e0cdd3154631.tar.bz2 px4-nuttx-07b5b43b323334b02fab26754dc6e0cdd3154631.zip |
Add some comments
Diffstat (limited to 'nuttx/include')
-rw-r--r-- | nuttx/include/wchar.h | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/nuttx/include/wchar.h b/nuttx/include/wchar.h index de087f3d1..90e3d4f12 100644 --- a/nuttx/include/wchar.h +++ b/nuttx/include/wchar.h @@ -106,6 +106,21 @@ typedef int wint_t; */ #if 0 /* Not used */ +/* Commented out because this is dangerous. This defines a type that would + * be internal to some wchar implementation. NuttX does not implement the + * wchar functions. Having this definition is a time bomb: If this header + * file is inadvertently included in code that interacts with an external + * library and if the definition of mbstate_t does not EXACTLY match the + * usage in that external library, then a potentially fatal error could + * occur. + * + * It is better to let the build error out due to the lack of the mbstate_t + * definition. At least that way, the problem can be avoided and the user + * can come up with some alternative way of dealing with the interfacing + * issue that does not require NuttX to be in lock-step with some external + * implementation. + */ + struct mbstate_s { int __fill[6]; @@ -113,6 +128,7 @@ struct mbstate_s typedef struct mbstate_s mbstate_t; #endif + /* FILE * As described in <stdio.h>. * |