Skip to content

lib: drop unused protocol headers #20093

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
Closed
vszakats wants to merge 17 commits into from
Closed

lib: drop unused protocol headers #20093

vszakats wants to merge 17 commits into from

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Dec 24, 2025 •
edited
Loading

  • drop unused http.h includes.
  • drop unused http1.h include.
  • drop unused http2.h includes.
  • vssh/ssh.h: drop unused vssh.h include.
  • urldata.h: drop unused protocol includes.
  • url: include smtp.h directly.
  • rtsp.h: include directly where used.
  • imap, smtp: drop redundant include, move another from .h to .c.

Verified with an all non-unity CI run.

  • http.h is included from urldata.h. Consider dropping from files including the latter. [SKIP for now]

@github-actions github-actions bot added the tests label Dec 24, 2025
@vszakats vszakats marked this pull request as draft December 24, 2025 23:33
@vszakats vszakats changed the title lib: drop unused http.h and http2.h includes lib: drop unused protocol headers Dec 25, 2025
@vszakats vszakats marked this pull request as ready for review December 25, 2025 01:03
@vszakats vszakats closed this in e78a466 Dec 25, 2025
@vszakats vszakats deleted the hdr20 branch December 25, 2025 11:00
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

build tests tidy-up

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats