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

add use-after-move rule to clang-tidy evergreen builder

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.7.0
    • Component/s: Internal Code
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Sprint:
      Execution Team 2020-08-10
    • Linked BF Score:
      40

      Description

      In SERVER-49229, we resolved a violation of the use-after-move rule in C++. Currently, our clang-tidy checker, introduced in SERVER-27984, does not enable this check. It would be beneficial to use the clang-tidy checker to detect other instances of the defect described in SERVER-49229.

        Attachments

          Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. SERVER-49876 Fix always-false if statement in CatalogCache

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

          Bug - A problem which impairs or prevents the functions of the product. SERVER-49229 use-after-move: 'options' argument in StorageEngineImpl constructor

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

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-49881 remove noop std::move from CollectionCatalog::setCollectionNamespace() rollback handler

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

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-27984 Add clang-tidy run in addition to lint

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

          Bug - A problem which impairs or prevents the functions of the product. SERVER-49997 Use after move in plan_cache_commands.cpp

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

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-50165 parallelize clang-tidy task

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

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-51240 Add clang tidy rules to prevent const return by value and (in declarations) const pass by value

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

          Web Link Clang Tools Documentation: bugprone-use-after-move

            Activity

              People

              Assignee:
              benety.goh Benety Goh
              Reporter:
              bynn.lee Bynn Lee (Inactive)
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: