[SERVER-64792] Consider renaming C++ header files to end with .hpp Created: 22/Mar/22 Updated: 07/Feb/23 Resolved: 07/Feb/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Andrew Morrow (Inactive) | Assignee: | [DO NOT ASSIGN] Backlog - Server Development Platform Team (SDP) (Inactive) |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Server Development Platform
|
| Participants: |
| Description |
|
Historically our header files have carried the .h suffix. Most toolchains consider that to be a C header, not C++. That hasn't been an issue in practice, for the most part, since the build system decides whether to build with C or C++ based on source file extensions, not header files. However, we do have tooling that wants to operate on header files directly sometimes, via things like clang-format or related IDE tooling. The use of the C-associated suffix can be misleading for those tools. We should evaluate whether we can bulk migrate our header files to a .hpp suffix. |
| Comments |
| Comment by Rui Liu [ 24/Mar/22 ] |
|
acm Oh my god, that should be |
| Comment by Githook User [ 23/Mar/22 ] |
|
Author: {'name': 'Rui Liu', 'email': 'rui.liu@mongodb.com', 'username': 'lriuui0x0'}Message: |