When I implemented scopes, there was a bug where the copy constructor would get called twice when creating and then adding a scope to the vector of all scopes.

This "double creation" meant that functions were cleaned-up at the wrong time.

This PR changes it such that frames are only created once. An option here would have been to define a move constructor and use emplace_back (rather than frames being a vector of pointers) -- @msoos do you have a preference here?

Signed-off-by: Andrew V. Jones [email protected]