Skip to content

compiler warning issues#714

Merged
pjfanning merged 4 commits into
apache:mainfrom
pjfanning:compiler-warnings
Aug 7, 2025
Merged

compiler warning issues#714
pjfanning merged 4 commits into
apache:mainfrom
pjfanning:compiler-warnings

Conversation

@pjfanning
Copy link
Copy Markdown
Member

  • reduces the issues that we silence with -Wcat
  • fixes a couple of issues
  • adds explicit nowarns and TODOs in places where we need to think about making changes
  • there are some other issues left behind that I hope to revisit

Copy link
Copy Markdown
Member

@He-Pin He-Pin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pjfanning pjfanning merged commit ee43056 into apache:main Aug 7, 2025
10 checks passed
@pjfanning pjfanning deleted the compiler-warnings branch August 7, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants