Report tasks on actual mapped products

aothms · aothms · commit 0b4718b3b5ce · 2023-12-05T21:12:09.000+01:00
diff --git a/src/ifcgeom/Iterator.h b/src/ifcgeom/Iterator.h
@@ -185,9 +185,9 @@ namespace IfcGeom {
 				tasks_.push_back(res);
 			}
 
-			std::vector<IfcUtil::IfcBaseClass*> products;
-			for (auto& r : reps) {
-				std::copy(r.products->begin(), r.products->end(), std::back_inserter(products));
+			size_t num_products = 0;
+			for (auto& r : tasks_) {
+				num_products += r.products.size();
 			}
 
 			/*
@@ -220,7 +220,7 @@ namespace IfcGeom {
 			}
 			*/
 
-			Logger::Notice("Created " + boost::lexical_cast<std::string>(tasks_.size()) + " tasks for " + boost::lexical_cast<std::string>(products.size()) + " products");
+			Logger::Notice("Created " + boost::lexical_cast<std::string>(tasks_.size()) + " tasks for " + boost::lexical_cast<std::string>(num_products) + " products");
 
 			if (tasks_.size() == 0) {
 				Logger::Warning("No representations encountered, aborting");

Original file line number	Diff line number	Diff line change
`@@ -185,9 +185,9 @@ namespace IfcGeom {`
`185`	`185`	`tasks_.push_back(res);`
`186`	`186`	`}`
`187`	`187`
`188`		`- std::vector<IfcUtil::IfcBaseClass*> products;`
`189`		`- for (auto& r : reps) {`
`190`		`- std::copy(r.products->begin(), r.products->end(), std::back_inserter(products));`
	`188`	`+ size_t num_products = 0;`
	`189`	`+ for (auto& r : tasks_) {`
	`190`	`+ num_products += r.products.size();`
`191`	`191`	`}`
`192`	`192`
`193`	`193`	`/*`
`@@ -220,7 +220,7 @@ namespace IfcGeom {`
`220`	`220`	`}`
`221`	`221`	`*/`
`222`	`222`
`223`		`- Logger::Notice("Created " + boost::lexical_cast<std::string>(tasks_.size()) + " tasks for " + boost::lexical_cast<std::string>(products.size()) + " products");`
	`223`	`+ Logger::Notice("Created " + boost::lexical_cast<std::string>(tasks_.size()) + " tasks for " + boost::lexical_cast<std::string>(num_products) + " products");`
`224`	`224`
`225`	`225`	`if (tasks_.size() == 0) {`
`226`	`226`	`Logger::Warning("No representations encountered, aborting");`