Re: [PR] Add missing c symbols to std namespace [nuttx]

2024-05-16 Thread via GitHub
acassis merged PR #12354: URL: https://github.com/apache/nuttx/pull/12354 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apac

Re: [PR] Add missing c symbols to std namespace [nuttx]

2024-05-16 Thread via GitHub
leducp commented on PR #12354: URL: https://github.com/apache/nuttx/pull/12354#issuecomment-2114247973 > > @nicolas71640 isn't __builtin_fpclassify and other something specific of GCC? Does it work for other C++ compilers as well? > > The similar code in math.h also reference __builti

Re: [PR] Add missing c symbols to std namespace [nuttx]

2024-05-15 Thread via GitHub
xiaoxiang781216 commented on PR #12354: URL: https://github.com/apache/nuttx/pull/12354#issuecomment-2114074375 > @nicolas71640 isn't __builtin_fpclassify and other something specific of GCC? Does it work for other C++ compilers as well? > The similar code in math.h also reference

Re: [PR] Add missing c symbols to std namespace [nuttx]

2024-05-15 Thread via GitHub
acassis commented on PR #12354: URL: https://github.com/apache/nuttx/pull/12354#issuecomment-2113314158 @nicolas71640 isn't __builtin_fpclassify and other something specific of GCC? Does it work for other C++ compilers as well? @xiaoxiang781216 and @masayuki2009 since you have more e

[PR] Add missing c symbols to std namespace [nuttx]

2024-05-15 Thread via GitHub
nicolas71640 opened a new pull request, #12354: URL: https://github.com/apache/nuttx/pull/12354 ## Summary Some symbols from libc were missing in std namespace. This PR add them, allowing better support of c++ ## Testing c++ project compilation ok. -- This is an automate