Chromium Code Reviews| Index: sdk/lib/_internal/compiler/implementation/inferrer/container_tracer.dart |
| =================================================================== |
| --- sdk/lib/_internal/compiler/implementation/inferrer/container_tracer.dart (revision 30542) |
| +++ sdk/lib/_internal/compiler/implementation/inferrer/container_tracer.dart (working copy) |
| @@ -141,7 +141,8 @@ |
| bool enableLengthTracking = true; |
| bool continueAnalyzing = true; |
| - |
| + bool checksGrowable = false; |
|
kasperl
2013/11/25 08:16:44
Is this always !enableLengthTracking? Should you u
ngeoffray
2013/11/25 08:33:31
Yes, it is always !enableLengthTracking. I combine
|
| + |
| static const int MAX_ANALYSIS_COUNT = 16; |
| final Setlet<Element> analyzedElements = new Setlet<Element>(); |
| @@ -249,9 +250,11 @@ |
| } |
| if (!doNotChangeLengthSelectorsSet.contains(selectorName)) { |
| enableLengthTracking = false; |
| + checksGrowable = true; |
| } |
| if (selectorName == 'length' && selector.isSetter()) { |
| enableLengthTracking = false; |
| + checksGrowable = true; |
| assignments.add(inferrer.types.nullType); |
| } |
| } else if (selector.isCall() |