[SERVER-24635] Remove duplicate fassert w/status function Created: 17/Jun/16 Updated: 06/Apr/18 Resolved: 08/Mar/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | None |
| Fix Version/s: | 3.7.3 |
| Type: | Task | Priority: | Minor - P4 |
| Reporter: | Scott Hernandez (Inactive) | Assignee: | Benety Goh |
| Resolution: | Done | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Sprint: | Repl 2018-03-26 | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
We have these two, with the same impl.:
We should also make sure when we remove one they are consistent with the StatusWith and no-trace versions. |
| Comments |
| Comment by Githook User [ 08/Mar/18 ] |
|
Author: {'email': 'benety@mongodb.com', 'name': 'Benety Goh', 'username': 'benety'}Message: This is redundant now that fassert supports both Status and StatusWith. |
| Comment by Benety Goh [ 07/Mar/18 ] |
|
At this point, all references to fassertStatusOK in master have been replaced by fassert. We will be removing fassertStatusOK from assert_util.h shortly. Until then, no new references to fassertStatusOK() should be introduced to the codebase. |
| Comment by Githook User [ 07/Mar/18 ] |
|
Author: {'email': 'benety@mongodb.com', 'name': 'Benety Goh', 'username': 'benety'}Message: |
| Comment by Githook User [ 07/Mar/18 ] |
|
Author: {'email': 'benety@mongodb.com', 'name': 'Benety Goh', 'username': 'benety'}Message: |