-
Notifications
You must be signed in to change notification settings - Fork 52
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
Make the library C89-compatible #173
Comments
The library is currently written with C99 assumed. What would it take to make it compile as C89? Apart from the comment style change? Also, do the C compiler on vxWorks 6.x really not support C99? |
its default option didn't support for double line comment |
And with a non-default option? Also, there are many other changes which would be necessary to make the library C89-compliant. Specifically, we would need to not use |
Currently only use block line comment is enough. |
The header is present, it's just the header with |
So, what you're asking is not C89 support, nor C90 support. Perhaps we can accommodate your compiler by some kind of CMakeLists.txt check, that would add an appropriate flag for |
Yeap, it's a -ansi option. I've seen a lot more compiling warning, I think achieve that would receive broader usage |
I think you've misread what I've written. The At any rate, it remains unclear to me what you are asking and for purpose. |
I am trying to use it within very old system such as VxWorks( and change the compiler option is not accepted) That's why I am trying to do c89 compat(Not allow |
I would consider a pull request which adds C89 support, if significant changes are #ifdef'ed ... |
There are other C90 limitations that the PR does not address. |
…C99-style single-line comments (`//`) Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
…C99-style single-line comments (`//`) Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
…C99-style single-line comments (`//`) Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Yeap, if change all would be a large , thank you for working on this, I am busy on other things |
For relative old but still usable OS such vxWorks 6.x, we want a cross-platform printf to handling printf things and use c89 syntax supported
The text was updated successfully, but these errors were encountered: