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

remove the verify() macro

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Internal Code

      Description

      The verify() macro is poorly designed because it throws an exception in release builds, and crashes in debug builds. In practice we can end up hitting verify()s at runtime fairly frequently, as indicated by jstestfuzz.

      As part of the 3.3 cycle we should remove the verify macro completely, replacing uses of it with invariant() or uassert() as appropriate.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              backlog-server-devtools DO NOT USE - Backlog - Dev Tools
              Reporter:
              adam.midvidy Adam Midvidy
              Participants:
              Votes:
              2 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: