Changeset 1199 for trunk/lib/Stats.cc

Timestamp:

Oct 4, 2010, 2:37:10 PM (12 years ago)

Author:

Peter Johansson

Message:

prefer operator+= rather than operator+

File:

• trunk/lib/Stats.cc (modified) (1 diff)

trunk/lib/Stats.cc

@@ -175 +175 @@
   void Stats::calc_all(void)
   {
-    // FIXME: we should use operator+=
-    SumVector init;
     for (int lt=0; lt <= 4; ++lt) {
       stats_[lt]["all"].clear();
-      stats_[lt]["all"] =
-        std::accumulate(stats_[lt].begin(), 
-                        stats_[lt].end(), init,
-                        PairValuePlus<std::string, SumVector>());
-    }
-
-    comment_or_copy_stats()["all"] = 
-      comment_stats()["all"] + copyright_stats()["all"];
-
-    total_stats()["all"] = code_stats()["all"] + 
-      comment_or_copy_stats()["all"] + other_stats()["all"];
+      for (std::map<std::string, SumVector>::iterator i = stats_[lt].begin();
+           i!=stats_[lt].end(); ++i) {
+        stats_[lt]["all"] += i->second;
+      }
+    }
+
+    comment_or_copy_stats()["all"] = comment_stats()["all"];
+    comment_or_copy_stats()["all"] += copyright_stats()["all"];
+
+    total_stats()["all"] = comment_or_copy_stats()["all"];
+    total_stats()["all"] += code_stats()["all"];
+    total_stats()["all"] += other_stats()["all"];
   }