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

Fix conflicting declaration of cpputest_strdup_location #1751

Merged
merged 1 commit into from Apr 6, 2023
Merged

Fix conflicting declaration of cpputest_strdup_location #1751

merged 1 commit into from Apr 6, 2023

Conversation

glennvl
Copy link

@glennvl glennvl commented Apr 6, 2023

I encountered the exact same issue as what was supposedly resolved in #1192, but for C++ code. What resolved it for me is adding #include <cstring> to StandardCLibrary.h.

Note that this appears to be a regression. 8dded9e is the last commit that builds without adding the extra header.

@coveralls
Copy link

Coverage Status

Coverage: 99.323%. Remained the same when pulling 7d97dd6 on glennvl:fix-strdup-conflicting-redeclaration into d51a055 on cpputest:master.

@basvodde basvodde merged commit 34f8c45 into cpputest:master Apr 6, 2023
74 checks passed
@glennvl glennvl deleted the fix-strdup-conflicting-redeclaration branch April 6, 2023 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants