Skip to content

gnutls: fix clang-tidy warning with !verbose#21060

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:gnutlswarn
Closed

gnutls: fix clang-tidy warning with !verbose#21060
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:gnutlswarn

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Mar 22, 2026

with
-DCURL_DISABLE_VERBOSE_STRINGS=ON -DCURL_USE_GNUTLS=ON -DCURL_CLANG_TIDY=ON.

lib/vtls/gtls.c:268:7: error: Value stored to 'strerr' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
  268 |       strerr = gnutls_strerror(rc);
      |       ^        ~~~~~~~~~~~~~~~~~~~

with:
`-DCURL_DISABLE_VERBOSE_STRINGS=ON -DCURL_USE_GNUTLS=ON -DCURL_CLANG_TIDY=ON`.

```
lib/vtls/gtls.c:268:7: error: Value stored to 'strerr' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
  268 |       strerr = gnutls_strerror(rc);
      |       ^        ~~~~~~~~~~~~~~~~~~~
```
@vszakats vszakats added the build label Mar 22, 2026
@github-actions github-actions bot added the TLS label Mar 22, 2026
@vszakats vszakats closed this in 757fd6a Mar 22, 2026
@vszakats vszakats deleted the gnutlswarn branch March 22, 2026 18:16
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