Homec4science

fixed weird memory bug in the default move constructor of CellBase

Authored by junge on May 16 2018, 17:32.

Description

fixed weird memory bug in the default move constructor of CellBase

Summary: turns out that the default move container in combination with clang segfaults under certain (unclear) cicumstances, because the move constructor of the optional appears to be busted in gcc 7.2. Copying it (K) instead of moving it fixes the issue, and since it is a reference, the cost is practically nil.

Test Plan: CI passes

Reviewers: afalsafi

Reviewed By: afalsafi

Differential Revision: https://c4science.ch/D188

Details

Committed
jungeMay 16 2018, 17:42
Pushed
jungeMay 16 2018, 17:42
Reviewer
afalsafi
Differential Revision
Restricted Differential Revision
Parents
rMUSPECTRE9bc97b7e7fa8: Refactoring field collections, FFT engines and solvers
Branches
Unknown
Tags
Unknown

Event Timeline

Restricted Application completed building Restricted Buildable.May 16 2018, 17:52