[CXX-165] Remove RecursiveMutex Created: 31/Mar/14  Updated: 04/May/14  Resolved: 31/Mar/14

Status: Closed
Project: C++ Driver
Component/s: None
Affects Version/s: None
Fix Version/s: legacy-0.8.0

Type: Task Priority: Major - P3
Reporter: Andrew Morrow (Inactive) Assignee: Andrew Morrow (Inactive)
Resolution: Done Votes: 0
Labels: legacy-cxx
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

The RecursiveMutex class in mutex.h is unused in the driver, but causes the threadlocal.h header to be needed as a user visible include which in turn would require the configuration #define's for threadlocal.h to be part of a future config.h

Since RecursiveMutex is unused, we should just remove it from mutex.h in our tree.

Longer term, we should consider whether we want to remove almost all of mutex.h and just standardize on boost::mutex in the driver. The hand rolled mutex classes are almost certainly not advantageous in the driver.



 Comments   
Comment by Andrew Morrow (Inactive) [ 31/Mar/14 ]

https://github.com/mongodb/mongo-cxx-driver/pull/44

Comment by Githook User [ 31/Mar/14 ]

Author:

{u'username': u'acmorrow', u'name': u'Andrew Morrow', u'email': u'acm@mongodb.com'}

Message: CXX-165 Remove recursive mutex to privatize thread local header

Also, a drive by removal of heap checking and FILE/LINE noise
from mutex.h
Branch: legacy
https://github.com/mongodb/mongo-cxx-driver/commit/3397a82c25354b84e35931b33430189dfbdb2963

Generated at Wed Feb 07 21:58:23 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.