Compiler warnings when building with GCC
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 68/100
Research direction
Reproduce the warnings with GCC 16 on Linux or mingw-w64 using -Wall -Wextra -Wpedantic. Inspect the reported anonymous structs in DirectXMath/Inc/DirectXMath.h and the unused variables in DirectXMath/Inc/DirectXMathMatrix.inl around line 1165. Done means the reported warnings are addressed while the library still builds successfully.
Written by the indexing model from the issue text.
Description
Building with GCC 16 on Linux or mingw-w64 using -Wall -Wextra -Wpedantic flags generates the following compiler warnings:
In file included from main.cpp:3:
DirectXMath/Inc/DirectXMath.h:845:13: warning: ISO C++ prohibits anonymous structs [-Wpedantic]
845 | {
| ^
DirectXMath/Inc/DirectXMath.h:903:13: warning: ISO C++ prohibits anonymous structs [-Wpedantic]
903 | {
| ^
DirectXMath/Inc/DirectXMath.h:975:13: warning: ISO C++ prohibits anonymous structs [-Wpedantic]
975 | {
| ^
DirectXMath/Inc/DirectXMath.h:1043:13: warning: ISO C++ prohibits anonymous structs [-Wpedantic]
1043 | {
| ^
In file included from DirectXMath/Inc/DirectXMath.h:2372:
DirectXMath/Inc/DirectXMathMatrix.inl: In function ‘bool DirectX::XMMatrixDecompose(XMVECTOR*, XMVECTOR*, XMVECTOR*, FXMMATRIX)’:
DirectXMath/Inc/DirectXMathMatrix.inl:1165:16: warning: variable ‘aa’ set but not used [-Wunused-but-set-variable=]
1165 | size_t aa, bb, cc;
| ^~
DirectXMath/Inc/DirectXMathMatrix.inl:1165:20: warning: variable ‘bb’ set but not used [-Wunused-but-set-variable=]
1165 | size_t aa, bb, cc;
| ^~
Clang doesn't seem to report any of these.
- Dominant language
- C++
- Stars
- 1.8k
- Forks
- 262
- Avg merge
- 3d 12h
- Merged PRs (30d)
- 5
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/DirectXMath
-
Difficulty 3/5 1-2 days Newbie friendliness 58/100
microsoft/DirectXMath#346 ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 25/100
microsoft/DirectXMath#328 ·
-
simplemath
Difficulty 5/5 Over a week Newbie friendliness 35/100
microsoft/DirectXMath#327 · 1 comment ·
-
Project feedback Open
Difficulty 5/5 Over a week Newbie friendliness 20/100
microsoft/DirectXMath#326 · 2 comments ·
-
arm64 maintenance
Difficulty 4/5 3-5 days Newbie friendliness 45/100
microsoft/DirectXMath#309 ·
All issues in microsoft/DirectXMath
Similar issues
-
AuTest Bug Tests
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
apache/trafficserver#13714 ·
-
bug build
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
facebookincubator/velox#19143 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
tenstorrent/tt-metal#57393 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
objectionary/eo-graphs#74 ·