Skip to content

Additional Compiler Warnings - #73

Merged
zfergus merged 4 commits into
mainfrom
warnings
Oct 24, 2023
Merged

Additional Compiler Warnings#73
zfergus merged 4 commits into
mainfrom
warnings

Conversation

@zfergus

@zfergus zfergus commented Oct 23, 2023

Copy link
Copy Markdown
Member

Add additional compiler warnings and resolve them to be warning-free.

@codecov

codecov Bot commented Oct 23, 2023

Copy link
Copy Markdown

Codecov Report

Merging #73 (4268d07) into main (5370866) will increase coverage by 0.00%.
The diff coverage is 97.95%.

@@           Coverage Diff           @@
##             main      #73   +/-   ##
=======================================
  Coverage   95.11%   95.12%           
=======================================
  Files          87       87           
  Lines        7251     7256    +5     
=======================================
+ Hits         6897     6902    +5     
  Misses        354      354           
Files Coverage Δ
src/ipc/broad_phase/aabb.cpp 100.00% <100.00%> (ø)
src/ipc/broad_phase/hash_grid.cpp 100.00% <100.00%> (ø)
src/ipc/broad_phase/hash_grid.hpp 100.00% <100.00%> (ø)
src/ipc/broad_phase/sweep_and_tiniest_queue.cpp 66.66% <100.00%> (ø)
src/ipc/broad_phase/sweep_and_tiniest_queue.hpp 100.00% <ø> (ø)
src/ipc/broad_phase/voxel_size_heuristic.cpp 95.58% <100.00%> (ø)
src/ipc/candidates/candidates.cpp 91.19% <100.00%> (ø)
src/ipc/candidates/edge_edge.cpp 82.00% <100.00%> (ø)
src/ipc/candidates/edge_face.cpp 100.00% <100.00%> (ø)
src/ipc/candidates/edge_vertex.cpp 85.10% <100.00%> (ø)
... and 20 more

@zfergus
zfergus merged commit 457b8a1 into main Oct 24, 2023
@zfergus
zfergus deleted the warnings branch October 24, 2023 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant