-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
Move from C++11 to C++17 (second try) #1196
Conversation
Almost there, just missing iOS. I see a lot of errors of the sort: "only available on iOS 11 or newer":
|
0128eac
to
aa949d8
Compare
This is an intermediate step in order to move to C++17.
We can't use it because the constructor is private. Also see: https://abseil.io/tips/134.
82539af
to
ad59b6b
Compare
Rebased on top of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! 🚀
@@ -238,25 +222,25 @@ class MAVLinkParameters { | |||
|
|||
bool set_as_same_type(const std::string& value_str) | |||
{ | |||
if (_value.is<uint8_t>()) { | |||
if (std::get_if<uint8_t>(&_value)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose _value.is
does not work with a variant, hence this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so. There is also std::get
which throws instead: https://en.cppreference.com/w/cpp/utility/variant/get
🎉 |
Previous try was #884.