We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Describe the bug memchr is called and then assigned to a const char * without a cast. This causes failures on certain compilers (llvm).
https://www.cplusplus.com/reference/cstring/memchr/
const void * memchr ( const void * ptr, int value, size_t num ); void * memchr ( void * ptr, int value, size_t num );
In file os-shared-common.h change
os-shared-common.h
const char *end = memchr(s, 0, maxlen);
to
const char *end = (const char *)memchr(s, 0, maxlen);
System observed on:
clang --version Apple clang version 12.0.0 (clang-1200.0.32.29) Target: x86_64-apple-darwin19.6.0 Thread model: posix InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
Reporter Info Sam Price - GSFC
The text was updated successfully, but these errors were encountered:
Fix nasa#1141, add typecast to memchr call
a6cf564
This function is documented as returning `void*`, and on some compilers this requires an explicit cast to `const char*` to avoid a warning.
Merge pull request #1142 from jphickey/fix-1141-memchr-cast
101adb4
Fix #1141, add typecast to memchr call
Fix nasa#1141, PipeName unset with debug event
a97a268
Do not print the pipe name in the debug event, just pipe ID.
Merge pull request nasa#1142 from jphickey/fix-1141-fixup-debug-event
7d311a7
Fix nasa#1141, pipe name unset
jphickey
Successfully merging a pull request may close this issue.
Describe the bug
memchr is called and then assigned to a const char * without a cast.
This causes failures on certain compilers (llvm).
https://www.cplusplus.com/reference/cstring/memchr/
In file
os-shared-common.h
change
to
System observed on:
Reporter Info
Sam Price - GSFC
The text was updated successfully, but these errors were encountered: