Merge pull request #781 from RosettaCommons/vmullig/fix_tom
Trying to fix the Mac issue with BridgeChainsTests unit test.
There were two instances in the code of something along the lines of:
std::string const & test_id = ids.top();
ids.pop();
<pass test_id to a function that uses it>
This doesn't work, because in plain English, it reads as:
-- Give me a const reference to the top element of the ids stack.
-- Remove the top element of the ids stack (so that my const reference is now invalid -- references are NOT owning pointers).
-- Do something with that const reference (resulting in an error).
The simple solution was to remove the ampersand, so that I'm creating a copy of the top element before destroying it.