[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 SERVER-64762 instead. Really sorry about that.

Comment by Githook User [ 23/Mar/22 ]

Author:

{'name': 'Rui Liu', 'email': 'rui.liu@mongodb.com', 'username': 'lriuui0x0'}

Message: SERVER-64792 Add test case for non-existent foreign collection
Branch: master
https://github.com/mongodb/mongo/commit/4ef1ac80482fadcead41739f5f0763bc37abe769

Generated at Thu Feb 08 06:01:09 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.