Skip to content

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