mosra / magnum-bindings

Bindings of the Magnum C++11 graphics engine into other languages
https://magnum.graphics
Other
19 stars 13 forks source link

Minor clang-tidy fixups in some of the "next" magnum-bindings #16

Closed Skylion007 closed 2 years ago

Skylion007 commented 2 years ago

Apply some minor perf/correctness improvement to the next-bindings:

codecov[bot] commented 2 years ago

Codecov Report

Merging #16 (f22d377) into next (732a0b7) will not change coverage. The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             next      #16   +/-   ##
=======================================
  Coverage   91.18%   91.18%           
=======================================
  Files          45       45           
  Lines        4709     4709           
=======================================
  Hits         4294     4294           
  Misses        415      415           
Impacted Files Coverage Δ
src/python/magnum/platform/cgl.cpp 61.53% <0.00%> (ø)
src/python/magnum/platform/glfw.cpp 36.84% <0.00%> (ø)
src/python/magnum/platform/glx.cpp 61.53% <0.00%> (ø)
src/python/magnum/gl.cpp 95.10% <100.00%> (ø)
src/python/magnum/platform/egl.cpp 76.92% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

mosra commented 2 years ago

Thank you!