From 6de0803debeca32942ed8ea745b68c2f7abe00db Mon Sep 17 00:00:00 2001 From: Brian Wilkerson Date: Thu, 6 Oct 2016 12:59:38 -0700 Subject: [PATCH] Clean up hints R=scheglov@google.com Review URL: https://codereview.chromium.org/2394733004 . --- pkg/analysis_server/lib/src/analysis_server.dart | 1 - pkg/analysis_server/tool/instrumentation/page/log_page.dart | 1 - pkg/analyzer/lib/src/context/context.dart | 4 ---- pkg/analyzer/lib/src/generated/engine.dart | 1 - pkg/analyzer/test/generated/analysis_context_factory.dart | 1 - pkg/analyzer/test/src/context/context_test.dart | 3 --- 6 files changed, 11 deletions(-) diff --git a/pkg/analysis_server/lib/src/analysis_server.dart b/pkg/analysis_server/lib/src/analysis_server.dart index b865b8e09c8..ae1bb0aa14e 100644 --- a/pkg/analysis_server/lib/src/analysis_server.dart +++ b/pkg/analysis_server/lib/src/analysis_server.dart @@ -38,7 +38,6 @@ import 'package:analyzer/src/generated/sdk.dart'; import 'package:analyzer/src/generated/source.dart'; import 'package:analyzer/src/generated/source_io.dart'; import 'package:analyzer/src/generated/utilities_general.dart'; -import 'package:analyzer/src/summary/package_bundle_reader.dart'; import 'package:analyzer/src/summary/pub_summary.dart'; import 'package:analyzer/src/task/dart.dart'; import 'package:analyzer/src/util/glob.dart'; diff --git a/pkg/analysis_server/tool/instrumentation/page/log_page.dart b/pkg/analysis_server/tool/instrumentation/page/log_page.dart index 11fb33c67d4..ad1194d3f8e 100644 --- a/pkg/analysis_server/tool/instrumentation/page/log_page.dart +++ b/pkg/analysis_server/tool/instrumentation/page/log_page.dart @@ -2,7 +2,6 @@ // for details. All rights reserved. Use of this source code is governed by a // BSD-style license that can be found in the LICENSE file. -import 'dart:collection'; import 'dart:math' as math; import '../log/log.dart'; diff --git a/pkg/analyzer/lib/src/context/context.dart b/pkg/analyzer/lib/src/context/context.dart index 3885e049a9c..d0a2104ba50 100644 --- a/pkg/analyzer/lib/src/context/context.dart +++ b/pkg/analyzer/lib/src/context/context.dart @@ -308,10 +308,6 @@ class AnalysisContextImpl implements InternalAnalysisContext { options.enableStrictCallChecks || this._options.enableGenericMethods != options.enableGenericMethods || this._options.enableSuperMixins != options.enableSuperMixins; - int cacheSize = options.cacheSize; - if (this._options.cacheSize != cacheSize) { - this._options.cacheSize = cacheSize; - } this._options.analyzeFunctionBodiesPredicate = options.analyzeFunctionBodiesPredicate; this._options.generateImplicitErrors = options.generateImplicitErrors; diff --git a/pkg/analyzer/lib/src/generated/engine.dart b/pkg/analyzer/lib/src/generated/engine.dart index 36c3ecd5460..fedd2e45d9f 100644 --- a/pkg/analyzer/lib/src/generated/engine.dart +++ b/pkg/analyzer/lib/src/generated/engine.dart @@ -1368,7 +1368,6 @@ class AnalysisOptionsImpl implements AnalysisOptions { */ AnalysisOptionsImpl.from(AnalysisOptions options) { analyzeFunctionBodiesPredicate = options.analyzeFunctionBodiesPredicate; - cacheSize = options.cacheSize; dart2jsHint = options.dart2jsHint; enableAssertInitializer = options.enableAssertInitializer; enableAssertMessage = options.enableAssertMessage; diff --git a/pkg/analyzer/test/generated/analysis_context_factory.dart b/pkg/analyzer/test/generated/analysis_context_factory.dart index e673e910e0b..e5b60f0a849 100644 --- a/pkg/analyzer/test/generated/analysis_context_factory.dart +++ b/pkg/analyzer/test/generated/analysis_context_factory.dart @@ -501,7 +501,6 @@ class AnalysisContextHelper { if (options == null) { options = new AnalysisOptionsImpl(); } - options.cacheSize = 256; context = AnalysisContextFactory.contextWithCoreAndOptions(options, resourceProvider: resourceProvider); } diff --git a/pkg/analyzer/test/src/context/context_test.dart b/pkg/analyzer/test/src/context/context_test.dart index e530e7fbbc6..3d8dc597b9c 100644 --- a/pkg/analyzer/test/src/context/context_test.dart +++ b/pkg/analyzer/test/src/context/context_test.dart @@ -2538,12 +2538,10 @@ class ClassTwo { void test_setAnalysisOptions() { AnalysisOptionsImpl options = new AnalysisOptionsImpl(); - options.cacheSize = 42; options.dart2jsHint = false; options.hint = false; context.analysisOptions = options; AnalysisOptions result = context.analysisOptions; - expect(result.cacheSize, options.cacheSize); expect(result.dart2jsHint, options.dart2jsHint); expect(result.hint, options.hint); } @@ -2733,7 +2731,6 @@ int a = 0;'''); int maxCacheSize = 4; AnalysisOptionsImpl options = new AnalysisOptionsImpl.from(context.analysisOptions); - options.cacheSize = maxCacheSize; context.analysisOptions = options; int sourceCount = maxCacheSize + 2; List sources = [];