Skip to content

clang-tidy: check bugprone-macro-parentheses, fix fallouts - #20647

Closed
vszakats wants to merge 13 commits into
curl:masterfrom
vszakats:ctidy10
Closed

vszakats wants to merge 13 commits into
curl:masterfrom
vszakats:ctidy10

Conversation

@vszakats

@vszakats vszakats commented Feb 20, 2026 •

Copy link
Copy Markdown
Member

Also:

  • lib/parsedate: avoid relying on side-effect of missing parentheses.
  • lib/http: drop redundant parentheses.
  • fix cases in headers missed by clang-tidy.

Ref: https://clang.llvm.org/extra/clang-tidy/checks/bugprone/macro-parentheses.html

@vszakats
vszakats marked this pull request as draft February 20, 2026 13:22
@vszakats vszakats changed the title clang-tidy: check bugprone-suspicious-realloc-usage, fix fallouts clang-tidy: check bugprone-macro-parentheses, fix fallouts Feb 20, 2026
@vszakats
vszakats marked this pull request as ready for review February 20, 2026 17:24
@vszakats
vszakats marked this pull request as draft February 20, 2026 23:03
@vszakats
vszakats marked this pull request as draft February 20, 2026 23:03
@vszakats
vszakats marked this pull request as ready for review February 20, 2026 23:43
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
Also:
- lib/parsedate: avoid relying on side-effect of missing parentheses.
- lib/http: drop redundant parentheses.
- fix cases in headers missed by clang-tidy.

Ref: https://clang.llvm.org/extra/clang-tidy/checks/bugprone/macro-parentheses.html

Closes curl#20647
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
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