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

Uniform #include style #39

Closed
bluescarni opened this issue Mar 10, 2017 · 2 comments
Closed

Uniform #include style #39

bluescarni opened this issue Mar 10, 2017 · 2 comments
Assignees
@bluescarni
Labels
enhancement
Milestone
pagmo 2.2

Comments

@bluescarni
Copy link
Member

At the moment we have a double style for the inclusion of pagmo headers within pagmo:

#include <pagmo/algorithm.hpp>

vs

#include "algorithm.hpp"

I would prefer if we switch eventually to the first style, which I like better because that is the same style someone using pagmo as C++ library would use, making things more consistent overall.
@bluescarni bluescarni self-assigned this Mar 10, 2017
@bluescarni bluescarni added this to the PaGMO 2.0 milestone Mar 10, 2017
@bluescarni bluescarni modified the milestones: pagmo 2.1, pagmo 2.0 Apr 8, 2017
darioizzo added a commit to darioizzo/pagmo2 that referenced this issue May 4, 2017
@darioizzo
[ci skip] Addressing esa#39
c7d7c6c
@darioizzo darioizzo mentioned this issue May 5, 2017
Merged
@bluescarni
Copy link
Member Author

@darioizzo is this finished with #96?

@darioizzo
Copy link
Member

Should indeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bluescarni bluescarni

Labels
enhancement
Projects
None yet
Milestone
pagmo 2.2
Development

No branches or pull requests
2 participants
@bluescarni @darioizzo