Event as std::variant #276
Closed
TobiasWallner
started this conversation in
General
Replies: 1 comment 3 replies
-
Std::variant it's c++17 and we would like to be c++11. We can use a union and refactor the event class for this. I tried to do this before but I focused on the more low level stuffs before polishing this |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
How about we change the class
Event
into anstd::variant
?The way I understand it is that only ever one of the member variables of the
Event
class will be used at the same time. So we might as well save some memory here by using avariant
.Beta Was this translation helpful? Give feedback.
All reactions