From 4f6fd961da510d9a2b6144e48272c68101376f0f Mon Sep 17 00:00:00 2001 From: "Yury G. Kudryashov" Date: Sat, 25 Feb 2012 13:20:57 +0400 Subject: [PATCH] Drop if(...) check because condition is always true GetLocation returns std::string::c_str() which is never NULL --- Source/cmLocalGenerator.cxx | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/Source/cmLocalGenerator.cxx b/Source/cmLocalGenerator.cxx index ffbeb4856..501fe614f 100644 --- a/Source/cmLocalGenerator.cxx +++ b/Source/cmLocalGenerator.cxx @@ -1902,15 +1902,8 @@ bool cmLocalGenerator::GetRealDependency(const char* inName, case cmTarget::SHARED_LIBRARY: case cmTarget::MODULE_LIBRARY: case cmTarget::UNKNOWN_LIBRARY: - { - // Get the location of the target's output file and depend on it. - if(const char* location = target->GetLocation(config)) - { - dep = location; - return true; - } - } - break; + dep = target->GetLocation(config); + return true; case cmTarget::UTILITY: case cmTarget::GLOBAL_TARGET: // A utility target has no file on which to depend. This was listed