Skip to content

tidy-up: fcntl.h includes #18782

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 2 commits into from
Closed

tidy-up: fcntl.h includes #18782

vszakats wants to merge 2 commits into from

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Sep 30, 2025 •
edited
Loading

  • drop from source files without obvious users.
  • include in curlx/fopen.h also for Windows.

Follow-up to 9678ff5 #18776

Tested OK with macOS/Windows example builds with the fcntl.h include deleted from curl_setup_once.h.

@testclutch

This comment was marked as outdated.

Sign in to view
@vszakats vszakats closed this in f97aa8d Sep 30, 2025
@vszakats vszakats deleted the trydrop branch September 30, 2025 19:57
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

tidy-up

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vszakats @testclutch