Skip to content

Conversation

redboltz
Copy link
Contributor

@redboltz redboltz commented Jul 2, 2020

Fixed warnings.

Fixed warnings.
@redboltz redboltz force-pushed the re_enable_werror branch from 9933fe0 to c8a0535 Compare July 2, 2020 07:15
@redboltz redboltz changed the title aPropagated CXXFLAGS. Propagated CXXFLAGS. Jul 2, 2020
@codecov-commenter
Copy link

Codecov Report

Merging #897 into cpp_master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           cpp_master     #897   +/-   ##
===========================================
  Coverage       86.43%   86.43%           
===========================================
  Files              72       72           
  Lines            5100     5100           
===========================================
  Hits             4408     4408           
  Misses            692      692           

@redboltz
Copy link
Contributor Author

redboltz commented Jul 2, 2020

I got https://github.com/msgpack/msgpack-c/runs/829664337?check_suite_focus=true#step:6:223 error.
I removed -fsanitize=undefined from linux build 3.

It seems that clang++ 32bit undefined sanitize build has known problem.
See https://www.google.com/search?q=undefined+reference+to+%60__mulodi4%27&oq=undefined+reference+to+%60__mulodi4%27

@redboltz redboltz merged commit eb1e7e6 into msgpack:cpp_master Jul 2, 2020
@redboltz redboltz added the C++ label Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants