Browse Source

cmTarget: Simplify CMP0023 message loop.

This method is only called if there is a mismatch and something to
print.  Remove intermediate container.
Stephen Kelly 10 years ago
parent
commit
a271f7f177
1 changed files with 5 additions and 15 deletions
  1. 5 15
      Source/cmTarget.cxx

+ 5 - 15
Source/cmTarget.cxx

@@ -1251,27 +1251,17 @@ bool cmTarget::PushTLLCommandTrace(TLLSignature signature)
 void cmTarget::GetTllSignatureTraces(std::ostringstream &s,
                                      TLLSignature sig) const
 {
-  std::vector<cmListFileContext> sigs;
+  const char *sigString = (sig == cmTarget::KeywordTLLSignature ? "keyword"
+                                                                : "plain");
+  s << "The uses of the " << sigString << " signature are here:\n";
   typedef std::vector<std::pair<TLLSignature, cmListFileContext> > Container;
+  cmLocalGenerator* lg = this->GetMakefile()->GetLocalGenerator();
   for(Container::const_iterator it = this->TLLCommands.begin();
       it != this->TLLCommands.end(); ++it)
     {
     if (it->first == sig)
       {
-      sigs.push_back(it->second);
-      }
-    }
-  cmLocalGenerator* lg = this->GetMakefile()->GetLocalGenerator();
-  if (!sigs.empty())
-    {
-    const char *sigString
-                        = (sig == cmTarget::KeywordTLLSignature ? "keyword"
-                                                                : "plain");
-    s << "The uses of the " << sigString << " signature are here:\n";
-    for(std::vector<cmListFileContext>::iterator it = sigs.begin();
-        it != sigs.end(); ++it)
-      {
-      cmListFileContext lfc = *it;
+      cmListFileContext lfc = it->second;
       lfc.FilePath = lg->Convert(lfc.FilePath, cmLocalGenerator::HOME);
       s << " * " << lfc << std::endl;
       }