[SERVER-31169] Use unique_ptr for MatchExpression sub-classes that store a child vector Created: 19/Sep/17 Updated: 06/Dec/22 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Justin Seyster | Assignee: | Backlog - Query Execution |
| Resolution: | Unresolved | Votes: | 1 |
| Labels: | query-44-grooming | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Query Execution
|
| Participants: |
| Description |
|
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. |