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

          duplicates

          Task - A task that needs to be done. SERVER-6317 Make assertions behave uniformly in debug and normal builds

          • Major - P3 - Major loss of function.
          • Open
          related to

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-25345 Replace all wasserts

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

            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: