[SERVER-43933] Utilize C++17 nested namespaces in logv2 Created: 10/Oct/19  Updated: 29/Oct/23  Resolved: 25/Mar/20

Status: Closed
Project: Core Server
Component/s: Logging
Affects Version/s: None
Fix Version/s: 4.7.0

Type: Task Priority: Major - P3
Reporter: Henrik Edin Assignee: Billy Donahue
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
Backwards Compatibility: Fully Compatible
Sprint: Service Arch 2020-04-06
Participants:

 Description   

Brought up in codereview: https://mongodbcr.appspot.com/512580001/

https://mongodbcr.appspot.com/512580001/diff/493720002/src/mongo/logv2/log_do...
src/mongo/logv2/log_domain_global.h:34: namespace mongo {
On 2019/10/09 17:16:36, henrik.edin wrote:
> On 2019/10/09 15:29:31, billy.donahue wrote:
> > Can we switch these over to be the C++17 deep "namespace mongo::logv2 {"
> blocks?
>
> Can we do that in a separate ticket? I will have to touch all the files and
this
> feels unrelated.

Acknowledged.



 Comments   
Comment by Githook User [ 25/Mar/20 ]

Author:

{'email': 'billy.donahue@mongodb.com', 'name': 'Billy Donahue', 'username': 'BillyDonahue'}

Message: SERVER-43933 open namespace mongo::logv2 C++17 syntax
Branch: master
https://github.com/mongodb/mongo/commit/599cabff5a9a40370d985addd8ef689c88211b18

Comment by Billy Donahue [ 23/Mar/20 ]

CR https://mongodbcr.appspot.com/566930042/

Generated at Thu Feb 08 05:04:30 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.