search

kainjow / Mustache

Mustache text templates for modern C++
Boost Software License 1.0
355 stars 49 forks source link

Remove unused lambda captures #22

Closed KitsuneRal closed 6 years ago

KitsuneRal commented 6 years ago

Some compilers don't like unused captures and whine about that.

codecov-io commented 6 years ago

Codecov Report

Merging #22 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   99.14%   99.14%           
=======================================
  Files           2        2           
  Lines        1283     1283           
=======================================
  Hits         1272     1272           
  Misses         11       11
Impacted Files Coverage Δ
mustache.hpp 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 14ed6be...01d71af. Read the comment docs.

kainjow commented 6 years ago

Thanks! Curious, what compilers warned about this?

kainjow commented 6 years ago

Ah looks like clang with Xcode 10 does. cool