Browse Source

Early return if there is no target.

The remainder of this method depends on the target existing (otherwise
the includes container would be empty), so make the code a little more
readable.
Stephen Kelly 13 years ago
parent
commit
0ef091d986
1 changed files with 7 additions and 4 deletions
  1. 7 4
      Source/cmLocalGenerator.cxx

+ 7 - 4
Source/cmLocalGenerator.cxx

@@ -1390,6 +1390,11 @@ void cmLocalGenerator::GetIncludeDirectories(std::vector<std::string>& dirs,
       }
     }
 
+  if(!target)
+    {
+    return;
+    }
+
   // Load implicit include directories for this language.
   std::string impDirVar = "CMAKE_";
   impDirVar += lang;
@@ -1407,10 +1412,8 @@ void cmLocalGenerator::GetIncludeDirectories(std::vector<std::string>& dirs,
 
   // Get the target-specific include directories.
   std::vector<std::string> includes;
-  if(target)
-    {
-    includes = target->GetIncludeDirectories();
-    }
+
+  includes = target->GetIncludeDirectories();
 
   // Support putting all the in-project include directories first if
   // it is requested by the project.