changeset 240a5a39e56f in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=240a5a39e56f
description:
        cleanup: Make use of types properly and make the loop a little more 
clear.

diffstat:

1 file changed, 11 insertions(+), 7 deletions(-)
src/base/range_ops.hh |   18 +++++++++++-------

diffs (38 lines):

diff -r 7cb9ce17a56f -r 240a5a39e56f src/base/range_ops.hh
--- a/src/base/range_ops.hh     Fri Jun 05 15:20:09 2009 -0700
+++ b/src/base/range_ops.hh     Fri Jun 05 17:01:19 2009 -0700
@@ -30,6 +30,7 @@
 
 #ifndef __BASE_RANGE_OPS_HH__
 #define __BASE_RANGE_OPS_HH__
+
 #include <list>
 #include <vector>
 
@@ -37,16 +38,19 @@
 
 template <class T>
 inline void
-FilterRangeList(std::vector<Range<T> > filter_list, std::list<Range<T> >
-        &range_list) {
-    typename std::list<Range<T> >::iterator i;
-    for (int x = 0; x < filter_list.size(); x++) {
-        for (i = range_list.begin(); i != range_list.end(); ) {
+FilterRangeList(std::vector<Range<T> > filter_list,
+                std::list<Range<T> > &range_list)
+{
+    typedef typename std::list<Range<T> > RangeList;
+
+    for (typename RangeList::size_type x = 0; x < filter_list.size(); x++) {
+        typename RangeList::iterator i = range_list.begin();
+        while (i != range_list.end()) {
             // Is the range within one of our filter ranges?
             if (filter_list[x] == i->start || filter_list[x] == i->end)
-                range_list.erase(i++);
+                i = range_list.erase(i);
             else
-                i++;
+                ++i;
         }
     }
 }
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to