Browse Source

Refactor: Drop useless assignments of `retval` before return

Alex Turbov 4 years ago
parent
commit
700abe7bca
1 changed files with 2 additions and 4 deletions
  1. 2 4
      Source/CPack/cmCPackDebGenerator.cxx

+ 2 - 4
Source/CPack/cmCPackDebGenerator.cxx

@@ -508,8 +508,7 @@ int cmCPackDebGenerator::PackageOnePack(std::string const& initialTopLevel,
   if (!this->ReadListFile("Internal/CPack/CPackDeb.cmake")) {
     cmCPackLogger(cmCPackLog::LOG_ERROR,
                   "Error while execution CPackDeb.cmake" << std::endl);
-    retval = 0;
-    return retval;
+    return 0;
   }
 
   { // Isolate globbing of binaries vs. dbgsyms
@@ -660,8 +659,7 @@ int cmCPackDebGenerator::PackageComponentsAllInOne(
   if (!this->ReadListFile("Internal/CPack/CPackDeb.cmake")) {
     cmCPackLogger(cmCPackLog::LOG_ERROR,
                   "Error while execution CPackDeb.cmake" << std::endl);
-    retval = 0;
-    return retval;
+    return 0;
   }
 
   cmsys::Glob gl;