GoogleCloudPlatform / functions-framework-cpp

Functions Framework for C++
Apache License 2.0
48 stars 24 forks source link

cleanup: fix GCC warnings with Wall #395

Closed coryan closed 1 year ago

coryan commented 1 year ago

I was too optimistic about the state of the code with GCC 13, while the code is clean with Clang, there was a useful warning emited by GCC 13, and a number of useless warnings too.

codecov[bot] commented 1 year ago

Codecov Report

Merging #395 (986303f) into main (6fabe85) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 986303f differs from pull request most recent head 89b5b24. Consider uploading reports for the commit 89b5b24 to get more accurate results

@@           Coverage Diff           @@
##             main     #395   +/-   ##
=======================================
  Coverage   89.40%   89.40%           
=======================================
  Files          97       97           
  Lines        3133     3133           
=======================================
  Hits         2801     2801           
  Misses        332      332           
Impacted Files Coverage Δ
examples/site/http_content/http_content.cc 97.91% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more