Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NuttX UINT32_C portability issue #630

Closed
PetervdPerk-NXP opened this issue Mar 26, 2020 · 2 comments
Closed

NuttX UINT32_C portability issue #630

PetervdPerk-NXP opened this issue Mar 26, 2020 · 2 comments
Labels
enhancement New feature or request Standards NuttX application interfaces must compy to standards

Comments

@PetervdPerk-NXP
Copy link
Contributor

In stdint.h UINT32_C has not been defined.
Per section 7.20.4.1 of the ISO C11 standard, these macros shall be defined by a compliant implementation of the standard library.

#if 0 /* REVISIT: Depends on architecture specific implementation */
#define INT8_C(x)           x
#define INT16_C(x)          x
#define INT32_C(x)          x ## l
#define INT64_C(x)          x ## ll

#define UINT8_C(x)          x
#define UINT16_C(x)         x
#define UINT32_C(x)         x ## ul
#define UINT64_C(x)         x ## ull
#endif

What would be a correct solution to define these macros for the corresponding CPU architecture?

@patacongo patacongo added enhancement New feature or request Standards NuttX application interfaces must compy to standards labels Mar 26, 2020
@PetervdPerk-NXP
Copy link
Contributor Author

It seems that is issue has been fixed by this commit 7758eb8, @xiaoxiang781216 that's correct right?

@xiaoxiang781216
Copy link
Contributor

Yes, thanks for closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Standards NuttX application interfaces must compy to standards
Projects
None yet
Development

No branches or pull requests

3 participants