[SERVER-68121] clang-tool boost::optional -> std::optional migration Created: 18/Jul/22 Updated: 19/Jan/24 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Billy Donahue | Assignee: | Backlog - Service Architecture |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Assigned Teams: |
Service Arch
|
||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Description |
|
Ticket created to track a skunkworks project in a persistent place. |
| Comments |
| Comment by Billy Donahue [ 09/Mar/23 ] |
|
Draft PR to integrate this check into the mongo repo's clang-tidy plugins framework. |
| Comment by Billy Donahue [ 22/Jul/22 ] |
|
That is interesting. Look how small that checker's source code is! I have a small match and replace function but I need to figure out how to run it on everything. And I wasn't using clang-tidy as a platform. I was using clang. |
| Comment by Alex Neben [ 22/Jul/22 ] |
|
I add the boost-use-to-string clang-tidy check in this variant It finds uses of "boost::lexical_cast<std::string>" and replaces them with "std::to_string". Just thought if you were doing replace boost stuff with the stl version and replacing the headers you might find this interesting. |
| Comment by Billy Donahue [ 22/Jul/22 ] |
|
Branch showing how to get away from boost/optional_io.hpp again: Rudimentary clangAST tool that does two things:
https://github.com/BillyDonahue/mongo_std_optional_migration |
| Comment by Billy Donahue [ 20/Jul/22 ] |
|
Older effort (bitrotted obviously) that I'm rehabilitating and updating. |