Skip to content
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

Convert all include guards to pragma once #50

Closed
SamVanheer opened this issue Nov 18, 2021 · 0 comments
Closed

Convert all include guards to pragma once #50

SamVanheer opened this issue Nov 18, 2021 · 0 comments
Assignees
@SamVanheer
Labels
enhancement New feature or request

Comments

@SamVanheer
Copy link
Collaborator

#pragma once is supported in MSVC, G++ and Clang and is preferred over macro-based include guards. It's simpler, smaller, doesn't pollute the global namespace, doesn't risk symbols conflicting and doesn't require changes when filenames change.
@SamVanheer SamVanheer added the enhancement New feature or request label Nov 18, 2021
@SamVanheer SamVanheer self-assigned this Nov 18, 2021
@SamVanheer SamVanheer reopened this Nov 18, 2021
SamVanheer added a commit to twhl-community/halflife-op4-updated that referenced this issue Nov 19, 2021
@SamVanheer
Convert Op4-specific headers to use pragma once
8aa9c01 
Resolves twhl-community/halflife-updated#50
LogicAndTrick pushed a commit to LogicAndTrick/halflife-updated that referenced this issue Dec 26, 2021
@SamVanheer @LogicAndTrick
Convert all include guards to pragma once
f6cc3ab 
Resolves twhl-community#50

(cherry picked from commit 0a3c17b)
LogicAndTrick pushed a commit to LogicAndTrick/halflife-updated that referenced this issue Dec 26, 2021
@SamVanheer @LogicAndTrick
Convert include guards that use !defined to pragma once
f238e0f 
Resolves twhl-community#50

(cherry picked from commit 58c150d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SamVanheer SamVanheer

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SamVanheer