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.

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

Code generation with preprocessor metaprogramming #163

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

vukics
Copy link

@vukics vukics commented Apr 24, 2020

Using Boost.Preprocessor for slicing operator to accept any number of arguments. (Cf. #44)

vukics and others added 3 commits July 3, 2018 12:41
@slayoo
Copy link
Member

slayoo commented Apr 24, 2020

We need to add checks for Boost to CMakeLists.txt, also .travis.yml and appveyor.yml files need an update to make continuous integration be "green".

And a question that was around for some time - are we OK to make Boost a required dependency or rather an option (like with serialisation and MPI)?

@citibeth @pguio @papadop ?

@vukics
Copy link
Author

vukics commented Apr 28, 2020

@slayoo
Yes, Boost dependency is an important decision to make.

Also, at present it’s only a proof of principle in this branch, a lot more code repetitions should be replaced with Boost.Preprocessor like this to make it worth adding this dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants