dynamorio icon indicating copy to clipboard operation
dynamorio copied to clipboard
verified publisher icon DynamoRIO

use C++ std::string comparison in droption.h to avoid link error

Open mxmssh opened this issue 8 years ago • 2 comments

droption.h includes both <string> and <string.h>. We should remove the <string.h> and replace the strcmp with a std::string comparison.

mxmssh avatar May 12 '17 12:05 mxmssh

The important piece of information is that you hit a linker error because of this, right? Could you paste that error here. It was in https://github.com/DynamoRIO/drmemory/pull/1982.

derekbruening avatar May 12 '17 13:05 derekbruening

Yes, it is here

/home/osboxes/drmemory_clean/build/dynamorio/ext/include/droption.h:40:0, from /home/osboxes/drmemory_clean/src/drltrace/drltrace_options.h:33, from /home/osboxes/drmemory_clean/src/drltrace/drltrace_frontend.cpp:44: /usr/include/string.h:369:26: error: previous declaration of ‘const char* strcasestr(const char*, const char*)’ with ‘C++’ linkage extern "C++" const char *strcasestr (const char *__haystack, ^ In file included from /home/osboxes/drmemory_clean/src/drltrace/drltrace_frontend.cpp:46:0: /home/osboxes/drmemory_clean/src/common/utils.h:1006:49: error: conflicts with new declaration with ‘C’ linkage

mxmssh avatar May 12 '17 13:05 mxmssh