Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix or silence compiler warnings #243

Open
SamVanheer opened this issue Nov 28, 2021 · 0 comments
Open

Fix or silence compiler warnings #243

SamVanheer opened this issue Nov 28, 2021 · 0 comments
Labels
enhancement New feature or request
Projects
Potential Upgrades

Comments

@SamVanheer
Copy link
Collaborator

Fix or silence compiler warnings. There are quite a number of them, though many are not relevant at this time.

Ideally no warnings should be silenced, but fixing all warnings is difficult since some of them require changes in how things are used, like enum class not supporting operator| by default.

In the short term warnings pertaining to buggy code should be fixed, other warnings should be fixed if it's simple enough, others should be silenced.
@SamVanheer SamVanheer self-assigned this Nov 28, 2021
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer
Silence MSVC compiler warnings C26495, C26451 and C26812
7258d55 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer
Overhaul vector types to use constexpr, simplify code
e27bc78 
Fix uninitialized variable warnings in Vector2D
Resolves #92
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer
FIx incorrect deletion of memory allocated with new[]
c9f8080 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer
Fix unannotated fallthrough warnings
bed2ed9 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer
Fix uses of sscanf not checking return values
3e9a8f0 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 29, 2021
@SamVanheer
Rework CSave & CRestore to eliminate null data pointers, clean up cod…
6606438 
…e a bit

Resolves #94
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 29, 2021
@SamVanheer
Fix uninitialized variable warnings that aren't bugs
70b19f7 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 29, 2021
@SamVanheer
Include Platform.h in VGUI.h to disable warnings
00fdb36 
#91
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 30, 2021
@SamVanheer
Zero-initialize arrays directly to silence "might not be terminated" …
6537f69 
…warning

#91
@SamVanheer SamVanheer transferred this issue from twhl-community/halflife-updated Dec 26, 2021
@SamVanheer SamVanheer added the enhancement New feature or request label Dec 26, 2021
@SamVanheer SamVanheer added this to To do in Upgrades via automation Dec 26, 2021
SamVanheer added a commit that referenced this issue Jan 21, 2022
@SamVanheer
Remove redundant pragma warning disable statements
bae95e1 
#243
@SamVanheer SamVanheer added this to To do in Code cleanup Jun 1, 2022
@SamVanheer SamVanheer removed this from To do in Upgrades Jun 1, 2022
@SamVanheer SamVanheer moved this from To do to In progress in Code cleanup Jun 1, 2022
SamVanheer added a commit that referenced this issue May 3, 2023
@SamVanheer
Fix code analysis warning in score panel code
b8fa509 
#243
@SamVanheer SamVanheer added this to In progress in Potential Upgrades Jun 17, 2023
@SamVanheer SamVanheer removed this from In progress in Code cleanup Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
Potential Upgrades
In progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SamVanheer