[SERVER-30223] Add a template for the individual timelib struct custom deleters Created: 19/Jul/17 Updated: 25/Jul/17 Resolved: 25/Jul/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Aggregation Framework |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Derick Rethans | Assignee: | Derick Rethans |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Sprint: | Query 2017-07-31 |
| Participants: |
| Description |
|
Right now, there are three custom deleters for timezone structs. Convert these to use a template so that code isn't duplicated. |
| Comments |
| Comment by Derick Rethans [ 25/Jul/17 ] |
|
After having had a good look at this, it turned out that this was more complicated than the benefits would be. The template was going to be needed in two classes, and for only three pointer types. The readability of the code would not improve, nor would it reduce the amount of code that would need maintenance. And hence I'm closing this. |