[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 ] |
| Comment by Githook User [ 31/Mar/14 ] |
|
Author: {u'username': u'acmorrow', u'name': u'Andrew Morrow', u'email': u'acm@mongodb.com'}Message: Also, a drive by removal of heap checking and FILE/LINE noise |