etr / libhttpserver

C++ library for creating an embedded Rest HTTP server (and more)
GNU Lesser General Public License v2.1
882 stars 184 forks source link

Fix-build-issues-clang-13 #291

Closed etr closed 1 year ago

etr commented 1 year ago

Identify the Bug

With the update of github actions to the latest version of ubuntu, we moved the library to build on clang-13 and gcc-10. Both have added additional warnings that this PR attempts to fix.

  1. The order in which libraries are included in deferred.cpp is wrong and mixes C includes with C++ includes.
  2. msan is mistakenly finding issues in system libraries.

Description of the Change

Re-order the includes in deferred.cpp. Force msan to ignore functions from system libraries.

Alternate Designs

None.

Possible Drawbacks

None.

Verification Process

Build process which includes testing.