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

use #pragma once #429

Closed
SunBlack opened this issue Jan 4, 2014 · 1 comment
Closed

use #pragma once #429

SunBlack opened this issue Jan 4, 2014 · 1 comment

Comments

@SunBlack
Copy link
Contributor

SunBlack commented Jan 4, 2014

I saw in the programming style guide that you still use #include guards. But all compilers supports support since years #pragma once, which has some advantages comparing to old #include guard (see here). So it would be nice, if this will be changed to modern variant.
@jspricke
Copy link
Member

jspricke commented Jan 4, 2014

Good idea, can you propose a pull request? Thanks!

@jspricke jspricke closed this as completed Jan 4, 2014
SunBlack pushed a commit to SunBlack/pcl that referenced this issue Nov 13, 2018
Replace include guards in all *.h-files by #pragma once
10507e8 
Fixes PointCloudLibrary#429
@SunBlack SunBlack mentioned this issue Nov 13, 2018
Merged
SunBlack pushed a commit to SunBlack/pcl that referenced this issue Nov 26, 2018
Replace include guards in all *.h-files by #pragma once
9f7a246 
Fixes PointCloudLibrary#429
@SunBlack SunBlack mentioned this issue Dec 12, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jspricke @SunBlack