Refactor removeRange.

pkg/unittest/lib/mock.dart

Index: pkg/unittest/lib/mock.dart
diff --git a/pkg/unittest/lib/mock.dart b/pkg/unittest/lib/mock.dart
index 730e4ff8a86a37240a5747411b50a410573df2b7..1c4471d84ee88c2bbcfd1f76f0f2b1027fd6dca0 100644
--- a/pkg/unittest/lib/mock.dart
+++ b/pkg/unittest/lib/mock.dart
@@ -513,7 +513,8 @@ class LogEntryList {
             actionMatcher.matches(entry, matchState)) {
           rtn.add(entry);
           if (destructive) {
-            logs.removeRange(i--, 1);
+            int startIndex = i--;

[Lasse Reichstein Nielsen, 2013/04/11 10:50:14]

Just: 
 i--;
 logs.removeRange(i, i + 1);
I don't like assigning and modifying in the same statement.

+            logs.removeRange(startIndex, startIndex + 1);
           }
         }
       }
@@ -621,7 +622,7 @@ class LogEntryList {
     int pos = findLogEntry(logFilter, 0, defaultPosition);
     if (inPlace) {
       if (pos < logs.length) {
-        logs.removeRange(pos, logs.length - pos);
+        logs.removeRange(pos, logs.length);
       }
       filter = description;
       return this;