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

wolfssl: silence compiler warning (MSVC 2019), simplify existing #16230

Closed
wants to merge 4 commits into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Feb 7, 2025

Seen with VS2019 and older versions:

lib\vtls\wolfssl.c(773): warning C4706: assignment within conditional expression

Ref: https://github.com/curl/curl/actions/runs/13190321645/job/36821938202?pr=16217#step:9:30

Also replace pragma suppression with this simpler method, and silence
checksrc where it complains about the extra != NULL this needs.

```
lib\vtls\wolfssl.c(773): warning C4706: assignment within conditional expression
```
https://github.com/curl/curl/actions/runs/13190321645/job/36821938202?pr=16217#step:9:30
Silence checksrc where it detects it as EQUALSNULL.
@github-actions github-actions bot added cmdline tool tests CI Continuous Integration labels Feb 7, 2025
@vszakats vszakats changed the title wolfssl: silence compiler warning (MSVC 2019) wolfssl: silence compiler warning (MSVC 2019 and older) Feb 7, 2025
@vszakats vszakats changed the title wolfssl: silence compiler warning (MSVC 2019 and older) wolfssl: silence compiler warning (MSVC 2019), simplify existing Feb 7, 2025
@vszakats vszakats closed this in 0b3afd1 Feb 7, 2025
@vszakats vszakats deleted the msvc4706 branch February 7, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant