Cleanup cmTarget source file list representation

This teaches cmTarget to use a set of cmSourceFile pointers to guarantee
unique insertion of source files in a target.  The order of insertion is
still preserved in the SourceFiles vector.

1 files changed, 16 insertions, 8 deletions

diff --git a/Source/cmTarget.cxx b/Source/cmTarget.cxx
index 8b4ac6f..26c0e39 100644
--- a/Source/cmTarget.cxx
+++ b/Source/cmTarget.cxx
@@ -1089,7 +1089,6 @@ private:
   cmGlobalGenerator* GlobalGenerator;
   std::queue<cmStdString> DependencyQueue;
   std::set<cmStdString> DependenciesQueued;
-  std::set<cmSourceFile*> TargetSources;
 
   void QueueOnce(std::string const& name);
   void QueueOnce(std::vector<std::string> const& names);
@@ -1120,9 +1119,6 @@ cmTargetTraceDependencies
     // Queue the dependencies of the source file in case they are
     // generated.
     this->QueueDependencies(*si);
-
-    // Track the sources already known to the target.
-    this->TargetSources.insert(*si);
     }
 
   // Queue the VS project file to check dependencies on the rule to
@@ -1156,10 +1152,7 @@ void cmTargetTraceDependencies::Trace()
       this->QueueDependencies(sf);
 
       // Make sure this file is in the target.
-      if(this->TargetSources.insert(sf).second)
-        {
-        this->Target->AddSourceFile(sf);
-        }
+      this->Target->AddSourceFile(sf);
       }
     }
 }
@@ -1331,6 +1324,21 @@ bool cmTarget::FindSourceFiles()
 }
 
 //----------------------------------------------------------------------------
+std::vector<cmSourceFile*> const& cmTarget::GetSourceFiles()
+{
+  return this->SourceFiles;
+}
+
+//----------------------------------------------------------------------------
+void cmTarget::AddSourceFile(cmSourceFile* sf)
+{
+  if(this->SourceFileSet.insert(sf).second)
+    {
+    this->SourceFiles.push_back(sf);
+    }
+}
+
+//----------------------------------------------------------------------------
 void cmTarget::AddSources(std::vector<std::string> const& srcs)
 {
   for(std::vector<std::string>::const_iterator i = srcs.begin();