Browse Source

cmFindPackageCommand: Drop dead code

The `cmFindPackageFileList` instances always constructed w/ the only
parameter. The boolean flag is always `true` (default value).

Also, `cmFindPackageCommand::InitialPass` adds an empty string
to the `cmFindPackageCommand::SearchPathSuffixes` vector. Meaning
that `cmFindPackageCommand::CheckDirectory()` gonna be called for
the suffix-less path, so `cmFindPackageFileList::Visit` may call
only `cmFindPackageCommand::SearchDirectory` and get the same result.
Alex Turbov 3 years ago
parent
commit
ef62f213f8
1 changed files with 2 additions and 8 deletions
  1. 2 8
      Source/cmFindPackageCommand.cxx

+ 2 - 8
Source/cmFindPackageCommand.cxx

@@ -2220,23 +2220,17 @@ bool cmFileListGeneratorBase::Consider(std::string const& fullPath,
 class cmFindPackageFileList : public cmFileList
 {
 public:
-  cmFindPackageFileList(cmFindPackageCommand* const fpc,
-                        bool const use_suffixes = true)
+  cmFindPackageFileList(cmFindPackageCommand* const fpc)
     : FPC(fpc)
-    , UseSuffixes(use_suffixes)
   {
   }
 
 private:
   bool Visit(std::string const& fullPath) override
   {
-    if (this->UseSuffixes) {
-      return this->FPC->SearchDirectory(fullPath);
-    }
-    return this->FPC->CheckDirectory(fullPath);
+    return this->FPC->SearchDirectory(fullPath);
   }
   cmFindPackageCommand* const FPC;
-  bool const UseSuffixes;
 };
 
 namespace {