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.

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

[Windows] Fix several build warnings #5804

Merged
merged 16 commits into from
Sep 11, 2024
Merged

Conversation

AenBleidd
Copy link
Member

No description provided.

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
…wxFONTWEIGHT_BOLD macro

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
- fix issue with variable shadowing
- fix int <-> size_t conversion warning
- fix size_t <-> unsigned int conversion warning

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
- fix variable shadowing warnings
- fix variable assignment in the condition expression warnings
- fix my_error_mgr structure variables order to preserve space.

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
…ng and int <-> UINT conversion warnings

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 10.49%. Comparing base (8cb5497) to head (3be93da).
Report is 19 commits behind head on master.

Files with missing lines Patch % Lines
lib/crypt_prog.cpp 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5804      +/-   ##
============================================
- Coverage     10.50%   10.49%   -0.02%     
  Complexity     1068     1068              
============================================
  Files           280      280              
  Lines         35972    36020      +48     
  Branches       8448     8444       -4     
============================================
  Hits           3780     3780              
- Misses        31798    31846      +48     
  Partials        394      394              
Files with missing lines Coverage Δ
api/gutil.cpp 0.00% <ø> (ø)
lib/common_defs.h 0.00% <ø> (ø)
lib/crypt_prog.cpp 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

@AenBleidd
Copy link
Member Author

@davidpanderson , please review

@AenBleidd AenBleidd marked this pull request as ready for review September 11, 2024 20:06
@davidpanderson davidpanderson merged commit 2b3cd21 into master Sep 11, 2024
146 of 147 checks passed
@davidpanderson davidpanderson deleted the vko_fix_win_warnings branch September 11, 2024 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

2 participants