Skip to content

fix compiler warnings flagged by MSVC on Windows #106

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
Merged
rdmark merged 2 commits into from
Jan 10, 2026
Merged

fix compiler warnings flagged by MSVC on Windows #106

rdmark merged 2 commits into from
Jan 10, 2026

Conversation

@rdmark
Copy link
Collaborator

@rdmark rdmark commented Jan 9, 2026

  • support for MSVC unused pragma in BSTR_UNUSED macro
  • explicit type cast of EOF macro value

@rdmark rdmark linked an issue Jan 9, 2026 that may be closed by this pull request
MSVC compiler warnings #103
Closed
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@github-actions
Copy link

github-actions bot commented Jan 9, 2026

File Coverage Lines Branches
All files 68% 73% 63% ✅
bstring/bstraux.c 55% 64% 46% ✅
bstring/bstrlib.c 73% 76% 69% ✅

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 3ad8a63

@rdmark rdmark merged commit a0ecb9a into main Jan 10, 2026
19 checks passed
@rdmark rdmark deleted the 103-msvc-compiler-warnings branch January 10, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

MSVC compiler warnings

2 participants

@rdmark