Browse Source

clang-tidy: fix `bugprone-parent-virtual-call` lints

Ben Boeckel 2 years ago
parent
commit
a805a897a8

+ 1 - 0
Source/cmGlobalVisualStudio10Generator.cxx

@@ -799,6 +799,7 @@ std::string cmGlobalVisualStudio10Generator::FindDevEnvCommand()
   }
   // Skip over the cmGlobalVisualStudio8Generator implementation because
   // we expect a real devenv and do not want to look for VCExpress.
+  // NOLINTNEXTLINE(bugprone-parent-virtual-call)
   return this->cmGlobalVisualStudio71Generator::FindDevEnvCommand();
 }
 

+ 1 - 0
Source/cmGlobalVisualStudio11Generator.cxx

@@ -243,6 +243,7 @@ bool cmGlobalVisualStudio11Generator::UseFolderProperty() const
   // Intentionally skip up to the top-level class implementation.
   // Folders are not supported by the Express editions in VS10 and earlier,
   // but they are in VS11 Express and above.
+  // NOLINTNEXTLINE(bugprone-parent-virtual-call)
   return cmGlobalGenerator::UseFolderProperty();
 }
 

+ 2 - 0
Source/cmGlobalVisualStudio8Generator.cxx

@@ -150,6 +150,7 @@ void cmGlobalVisualStudio8Generator::Configure()
 
 bool cmGlobalVisualStudio8Generator::UseFolderProperty() const
 {
+  // NOLINTNEXTLINE(bugprone-parent-virtual-call)
   return IsExpressEdition() ? false : cmGlobalGenerator::UseFolderProperty();
 }
 
@@ -381,6 +382,7 @@ bool cmGlobalVisualStudio8Generator::ComputeTargetDepends()
 {
   // Skip over the cmGlobalVisualStudioGenerator implementation!
   // We do not need the support that VS <= 7.1 needs.
+  // NOLINTNEXTLINE(bugprone-parent-virtual-call)
   return this->cmGlobalGenerator::ComputeTargetDepends();
 }