blob: e3025d22e291cef3f249e6b3275ad0e270d6a197 [file]
// Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
// 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 'package:analyzer/analysis_rule/analysis_rule.dart';
import 'package:analyzer/analysis_rule/rule_context.dart';
import 'package:analyzer/analysis_rule/rule_visitor_registry.dart';
import 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/dart/ast/visitor.dart';
import 'package:analyzer/dart/element/type.dart';
import 'package:analyzer/error/error.dart';
import '../analyzer.dart';
import '../diagnostic.dart' as diag;
import '../extensions.dart';
const _desc = r'Avoid annotating with `dynamic` when not required.';
class AvoidAnnotatingWithDynamic extends AnalysisRule {
new()
: super(name: LintNames.avoid_annotating_with_dynamic, description: _desc);
@override
DiagnosticCode get diagnosticCode => diag.avoidAnnotatingWithDynamic;
@override
void registerNodeProcessors(
RuleVisitorRegistry registry,
RuleContext context,
) {
var visitor = _Visitor(this);
registry.addFieldFormalParameter(this, visitor);
registry.addRegularFormalParameter(this, visitor);
registry.addSuperFormalParameter(this, visitor);
}
}
class _Visitor(final AnalysisRule rule) extends SimpleAstVisitor<void> {
@override
void visitFieldFormalParameter(FieldFormalParameter node) {
_checkNode(node, node.type);
}
@override
void visitRegularFormalParameter(RegularFormalParameter node) {
_checkNode(node, node.type);
}
@override
void visitSuperFormalParameter(SuperFormalParameter node) {
_checkNode(node, node.type);
}
void _checkNode(FormalParameter node, TypeAnnotation? type) {
if (node.inAugmentation) return;
if (type is NamedType && type.type is DynamicType) {
rule.reportAtNode(node);
}
}
}
extension on AstNode {
bool get inAugmentation {
AstNode? target = this;
while (target != null) {
if (target.isAugmentation) return true;
if (target is Block) return false;
if (target is Declaration) return false;
target = target.parent;
}
return false;
}
}