Skip to content

Commit af5e3aa

Browse files
srawlinsCommit Queue
authored andcommitted
analyzer: Remove some exports from rule_context.dart
Until recently, everything in rule_context.dart was found in linter.dart. That file was broken up [1]. That file previously had a lot of exports; it was a sort of barrel file for lint rules. In order to make that change "minimal," (it was still huge), I retained some of the exports. Now we're in the process of removing those exports. In this change, I remove the exports for `rule_state.dart` and `linter_visitor.dart`. This doesn't affect too many files. [1]: 438b560 Change-Id: I83110b22a07186560f6c853f033e070ac050ecd1 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/445123 Commit-Queue: Samuel Rawlins <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]>
1 parent 922aff8 commit af5e3aa

File tree

6 files changed

+6
-8
lines changed

6 files changed

+6
-8
lines changed

pkg/analysis_server/test/src/services/correction/fix/analysis_options/remove_lint_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5+
import 'package:analyzer/analysis_rule/rule_state.dart';
56
import 'package:analyzer/error/error.dart';
6-
import 'package:analyzer/src/analysis_rule/rule_context.dart';
77
import 'package:analyzer/src/lint/linter.dart';
88
import 'package:analyzer/src/test_utilities/lint_registration_mixin.dart';
99
import 'package:linter/src/rules.dart';

pkg/analysis_server_plugin/test/src/lint_rules.dart

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5+
import 'package:analyzer/analysis_rule/analysis_rule.dart';
56
import 'package:analyzer/analysis_rule/rule_context.dart';
67
import 'package:analyzer/analysis_rule/rule_visitor_registry.dart';
78
import 'package:analyzer/dart/ast/ast.dart';
89
import 'package:analyzer/dart/ast/visitor.dart';
910
import 'package:analyzer/error/error.dart';
10-
import 'package:analyzer/src/analysis_rule/rule_context.dart';
1111

1212
class NoBoolsRule extends AnalysisRule {
1313
static const LintCode code = LintCode('no_bools', 'No bools message');
@@ -56,7 +56,8 @@ class NoDoublesWarningRule extends AnalysisRule {
5656
DiagnosticCode get diagnosticCode => code;
5757

5858
@override
59-
void registerNodeProcessors(NodeLintRegistry registry, RuleContext context) {
59+
void registerNodeProcessors(
60+
RuleVisitorRegistry registry, RuleContext context) {
6061
var visitor = _NoDoublesVisitor(this);
6162
registry.addDoubleLiteral(this, visitor);
6263
}

pkg/analyzer/lib/src/analysis_rule/rule_context.dart

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,6 @@ import 'package:analyzer/dart/element/type_system.dart';
1111
import 'package:analyzer/workspace/workspace.dart';
1212

1313
export 'package:analyzer/analysis_rule/analysis_rule.dart';
14-
export 'package:analyzer/analysis_rule/rule_state.dart'
15-
show dart2_12, dart3, dart3_3, RuleState;
16-
export 'package:analyzer/src/lint/linter_visitor.dart' show NodeLintRegistry;
1714

1815
/// Returns whether [filePath] is in the top-level `lib` directory in [package].
1916
bool _isInLibDir(String? filePath, WorkspacePackage? package) {

pkg/analyzer/lib/src/lint/options_rule_validator.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5+
import 'package:analyzer/analysis_rule/rule_state.dart';
56
import 'package:analyzer/diagnostic/diagnostic.dart';
67
import 'package:analyzer/error/listener.dart';
78
import 'package:analyzer/src/analysis_options/error/option_codes.dart';

pkg/analyzer/test/src/options/options_rule_validator_test.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import 'package:analyzer/analysis_rule/rule_state.dart';
66
import 'package:analyzer/error/error.dart';
77
import 'package:analyzer/error/listener.dart';
88
import 'package:analyzer/src/analysis_options/error/option_codes.dart';
9-
import 'package:analyzer/src/analysis_rule/rule_context.dart';
109
import 'package:analyzer/src/lint/linter.dart';
1110
import 'package:analyzer/src/lint/options_rule_validator.dart';
1211
import 'package:analyzer/src/string_source.dart';

pkg/analyzer_testing/lib/analysis_rule/analysis_rule.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@
44

55
import 'dart:convert' show json;
66

7+
import 'package:analyzer/analysis_rule/analysis_rule.dart';
78
import 'package:analyzer/analysis_rule/pubspec.dart';
89
import 'package:analyzer/diagnostic/diagnostic.dart';
910
import 'package:analyzer/error/error.dart';
1011
import 'package:analyzer/error/listener.dart';
1112
import 'package:analyzer/source/file_source.dart';
12-
import 'package:analyzer/src/analysis_rule/rule_context.dart'; // ignore: implementation_imports
1313
import 'package:analyzer/src/lint/pub.dart'; // ignore: implementation_imports
1414
import 'package:analyzer/src/lint/registry.dart'; // ignore: implementation_imports
1515
import 'package:analyzer_testing/src/analysis_rule/pub_package_resolution.dart';

0 commit comments

Comments
 (0)