Skip to content

Add flag BOOST_VARIANT_USE_RELAXED_GET_BY_DEFAULT.#3531

Closed
ealmansi wants to merge 1 commit intoargotorg:developfrom
ealmansi:fix-3530
Closed

Add flag BOOST_VARIANT_USE_RELAXED_GET_BY_DEFAULT.#3531
ealmansi wants to merge 1 commit intoargotorg:developfrom
ealmansi:fix-3530

Conversation

@ealmansi
Copy link
Contributor

@ealmansi ealmansi commented Feb 15, 2018

Fixes #3530

Echoing from the linked issue: I don't have personal experience so as to judge whether this solution is reasonable or somewhat of a hack.

@chriseth
Copy link
Contributor

I would rather avoid removing what looks like safety features. Could you please check if changing the line from

FunctionalInstruction const& instruction = boost::get<FunctionalInstruction const&>(_expr);

to

FunctionalInstruction const& instruction = boost::get<FunctionalInstruction>(_expr);

fixes it?

@leonardoalt
Copy link

I also had this problem and confirm that applying that change (removing const &) to that and 3 other occurrences leads to compilation working again.

@chriseth
Copy link
Contributor

Closing this in favour of #3536

@chriseth chriseth closed this Feb 16, 2018
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.

3 participants