Skip to content

curl_setup.h: drop extra header guard for internal include#20544

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

curl_setup.h: drop extra header guard for internal include#20544
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:dropguard

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Feb 8, 2026

The included local header starts with this same guard. The original
commit added it for fixing VMS builds along with many other changes, but
without mention of this specific one in the commit message.

curl_setup.h is included once, which includes curl_setup_once.h
once, even if the latter wouldn't have it's own guard.

Ref: 25f3514

The header itself starts with this guard. The original commit added it
for fixing VMS builds along with many other changes, but without mention
of this specific change in the commit message.

`curl_setup.h` is included once, which includes `curl_setup_once.h`
once, even if the latter wouldn't have it's own guard.

Ref: 25f3514
@vszakats vszakats changed the title curl_setup.h: drop extra header guard for curl include curl_setup.h: drop extra header guard for internal include Feb 8, 2026
@vszakats vszakats closed this in 2c0019b Feb 8, 2026
@vszakats vszakats deleted the dropguard branch February 8, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant