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

Stand-alone with C++17 std library, no boost #78

Open
elefthei opened this issue Apr 23, 2018 · 7 comments
Open

Stand-alone with C++17 std library, no boost #78

elefthei opened this issue Apr 23, 2018 · 7 comments
Assignees

Comments

@elefthei
Copy link

elefthei commented Apr 23, 2018

I was wondering if there's any plan to making Mach7 stand-alone with the introduction of std::variant and std::visit in C++, in comparison to boost::variant currently used on master. I briefly searched open issues but maybe I missed the relevant conversation.

Is there any additional dependencies on boost except variant?

@solodon4 solodon4 self-assigned this Apr 24, 2018
@solodon4
Copy link
Owner

I'll try to find time to write the necessary adapter for std::variant in the next couple of days. Thanks for the reminder!

@elefthei
Copy link
Author

elefthei commented Apr 24, 2018

Thanks @solodon4 for the fast response! Two more cents from me, an unimportant one first;
In addition to std::variant, std::any made it into C++17 which could replace boost::any which should be all the need for boost at this point.

And a more important one; clang 3.9-4 might be required to support these features and--std=gnu++17
https://clang.llvm.org/cxx_status.html

Unfortunately, some experiments I run gave me a very bad performance in generated executables with later versions of clang (clang++-6.0 is what I tried). I would be interested in helping debug and fix these runtime problems with later clang.

@solodon4
Copy link
Owner

I looked into boost::any a while back in my discussion with Andrzej and that one will require a bit more work to be correct. I will concentrate on std::variant first as that one should be straightforward.

@elfprince13
Copy link
Contributor

Curious if there's any update on this front as my project which uses Mach7 just updated to C++17.

@solodon4
Copy link
Owner

No updates yet unfortunately - was busy with other projects. I left Microsoft though recently and while I'm on sort of sabbatical I should have a better chance to look at this. Will post here with updates once i do

@elefthei
Copy link
Author

elefthei commented Nov 21, 2019 via email

@sgf
Copy link

sgf commented Jul 2, 2020

Please don't mind, I just want to care.
Is there any progress on remove boost dependent?

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

No branches or pull requests

4 participants