ElemMatchValueMatchExpression and ListOfMatchExpression each store a vector of MatchExpression raw pointers that have to be cleaned up manually in their destructors. We'd like to replace these each with a vector of std::unique_ptr<MatchExpression> to formalize their ownership and improve our C++ hygiene.