Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-24635

Remove duplicate fassert w/status function

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor - P4
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.7.3
    • Component/s: Internal Code
    • Labels:
    • Backwards Compatibility:
      Fully Compatible
    • Sprint:
      Repl 2018-03-26

      Description

      We have these two, with the same impl.:

      inline void fassertStatusOK(int msgid, const Status& s);
      		 
      inline void fassert(int msgid, const Status& status);

      We should also make sure when we remove one they are consistent with the StatusWith and no-trace versions.

        Attachments

          Issue Links

          related to

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-33659 add no-trace variant fassertStatusOK that accepts StatusWith

          • Major - P3 - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-34364 add Status support to invariant()

          • Minor - P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed

            Activity

              People

              Assignee:
              benety.goh Benety Goh
              Reporter:
              scotthernandez Scott Hernandez
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: