[cmake-commits] martink committed cmGlobalUnixMakefileGenerator3.cxx 1.102 1.103

[cmake-commits at cmake.org]

Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv22845

Modified Files:
	cmGlobalUnixMakefileGenerator3.cxx 
Log Message:
BUG: fix progress for ENCLUDE_FORM_ALL cases using new project to target map. Only fixes it for the top level all target


Index: cmGlobalUnixMakefileGenerator3.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmGlobalUnixMakefileGenerator3.cxx,v
retrieving revision 1.102
retrieving revision 1.103
diff -u -d -r1.102 -r1.103
--- cmGlobalUnixMakefileGenerator3.cxx	10 Apr 2007 12:49:42 -0000	1.102
+++ cmGlobalUnixMakefileGenerator3.cxx	12 Apr 2007 18:21:58 -0000	1.103
@@ -912,40 +912,29 @@
   return result;
 }
 
-unsigned long cmGlobalUnixMakefileGenerator3::
-GetNumberOfProgressActionsInAll(cmLocalUnixMakefileGenerator3 *lg)
+unsigned long cmGlobalUnixMakefileGenerator3
+::GetNumberOfProgressActionsInAll(cmLocalUnixMakefileGenerator3 *lg)
 {
   unsigned long result = 0;
 
   // for every target in the top level all
   if (!lg->GetParent())
     {
-    // loop over the generators and targets
-    unsigned int i;
-    cmLocalUnixMakefileGenerator3 *lg3;
-    for (i = 0; i < this->LocalGenerators.size(); ++i)
+    // use the new project to target map   
+    std::set<cmTarget*> &targets = 
+      this->ProjectToTargetMap[lg->GetMakefile()->GetProjectName()];
+    std::set<cmTarget*>::iterator t = targets.begin();
+    for(; t != targets.end(); ++t)
       {
-      lg3 = static_cast<cmLocalUnixMakefileGenerator3 *>
-        (this->LocalGenerators[i]);
-      // for each target Generate the rule files for each target.
-      cmTargets& targets = lg3->GetMakefile()->GetTargets();
-      for(cmTargets::iterator t = targets.begin(); t != targets.end(); ++t)
-        {
-        if((t->second.GetType() == cmTarget::EXECUTABLE) ||
-           (t->second.GetType() == cmTarget::STATIC_LIBRARY) ||
-           (t->second.GetType() == cmTarget::SHARED_LIBRARY) ||
-           (t->second.GetType() == cmTarget::MODULE_LIBRARY) ||
-           (t->second.GetType() == cmTarget::UTILITY))
-          {
-          if (!t->second.GetPropertyAsBool("EXCLUDE_FROM_ALL"))
-            {
-            std::vector<int> &progFiles = lg3->ProgressFiles[t->first];
-            result += static_cast<unsigned long>(progFiles.size());
-            }
-          }
-        }
+      cmTarget* target = *t;
+      cmLocalUnixMakefileGenerator3 *lg3 = 
+        static_cast<cmLocalUnixMakefileGenerator3 *>
+        (target->GetMakefile()->GetLocalGenerator());
+      std::vector<int> &progFiles = lg3->ProgressFiles[target->GetName()];
+      result += static_cast<unsigned long>(progFiles.size());
       }
     }
+  // for subdirectories
   else
     {
     std::deque<cmLocalUnixMakefileGenerator3 *> lg3Stack;