3f7df09ddb
We rename `AnalyzerOptions` to `AnalysisFileOptions`, and change it's comment from "`analyzer` analysis options constants." to avoid anyone thinking it is specifying options for the `analyzer` section. Additionally I privatize some of the static values that are only present for validation - they are not meant to be used outside of `options.dart`: `AnalyzerSection._trueOrFalse`, `AnalyzerSection._options`, `AnalyzerSection._strongModeOptions`, `AnalyzerSection._languageOptions`, `LinterSection._options`. I also rename `_TopLevelAnalyzerOptionsValidator` to `_AnalyzerTopLevelOptionsValidator` to align with `_LinterTopLevelOptionsValidator`. Change-Id: I4f5735d8bda8edbaf066387b473c05969c3ba7d5 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/395060 Commit-Queue: Samuel Rawlins <srawlins@google.com> Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
108 lines
3.5 KiB
Dart
108 lines
3.5 KiB
Dart
// Copyright (c) 2015, 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/error/error.dart';
|
|
import 'package:analyzer/src/generated/engine.dart';
|
|
import 'package:analyzer/src/task/options.dart';
|
|
import 'package:yaml/yaml.dart';
|
|
|
|
/// String identifiers mapped to associated severities.
|
|
const Map<String, ErrorSeverity> severityMap = {
|
|
'error': ErrorSeverity.ERROR,
|
|
'info': ErrorSeverity.INFO,
|
|
'warning': ErrorSeverity.WARNING
|
|
};
|
|
|
|
/// Error processor configuration derived from analysis (or embedder) options.
|
|
class ErrorConfig {
|
|
/// The processors in this config.
|
|
final List<ErrorProcessor> processors = <ErrorProcessor>[];
|
|
|
|
/// Create an error config for the given error code map.
|
|
/// For example:
|
|
/// new ErrorConfig({'missing_return' : 'error'});
|
|
/// will create a processor config that turns `missing_return` warnings into
|
|
/// errors.
|
|
ErrorConfig(YamlNode? codeMap) {
|
|
if (codeMap is YamlMap) {
|
|
_processMap(codeMap);
|
|
}
|
|
}
|
|
|
|
void _processMap(YamlMap codes) {
|
|
codes.nodes.forEach((k, v) {
|
|
if (k is YamlScalar && v is YamlScalar) {
|
|
var code = k.value;
|
|
if (code is! String) return;
|
|
|
|
code = code.toUpperCase();
|
|
var action = v.value.toString().toLowerCase();
|
|
if (AnalysisOptionsFile.ignoreSynonyms.contains(action)) {
|
|
processors.add(ErrorProcessor.ignore(code));
|
|
} else {
|
|
var severity = severityMap[action];
|
|
if (severity != null) {
|
|
processors.add(ErrorProcessor(code, severity));
|
|
}
|
|
}
|
|
}
|
|
});
|
|
}
|
|
}
|
|
|
|
/// Process errors by filtering or changing associated [ErrorSeverity].
|
|
class ErrorProcessor {
|
|
/// The code name of the associated error.
|
|
final String code;
|
|
|
|
/// The desired severity of the processed error.
|
|
///
|
|
/// If `null`, this processor will "filter" the associated error code.
|
|
final ErrorSeverity? severity;
|
|
|
|
/// Create an error processor that assigns errors with this [code] the
|
|
/// given [severity].
|
|
///
|
|
/// If [severity] is `null`, matching errors will be filtered.
|
|
ErrorProcessor(this.code, [this.severity]);
|
|
|
|
/// Create an error processor that ignores the given error by [code].
|
|
factory ErrorProcessor.ignore(String code) => ErrorProcessor(code);
|
|
|
|
/// The string that unique describes the processor.
|
|
String get description => '$code -> ${severity?.name}';
|
|
|
|
/// Check if this processor applies to the given [error].
|
|
///
|
|
/// Note: [code] is normalized to uppercase; `errorCode.name` for regular
|
|
/// analysis issues uses uppercase; `errorCode.name` for lints uses lowercase.
|
|
bool appliesTo(AnalysisError error) =>
|
|
code == error.errorCode.name ||
|
|
code == error.errorCode.name.toUpperCase();
|
|
|
|
@override
|
|
String toString() => "ErrorProcessor[code='$code', severity=$severity]";
|
|
|
|
/// Return an error processor associated in the [analysisOptions] for the
|
|
/// given [error], or `null` if none is found.
|
|
static ErrorProcessor? getProcessor(
|
|
AnalysisOptions? analysisOptions, AnalysisError error) {
|
|
if (analysisOptions == null) {
|
|
return null;
|
|
}
|
|
|
|
// Let the user configure how specific errors are processed.
|
|
List<ErrorProcessor> processors = analysisOptions.errorProcessors;
|
|
|
|
// Add the strong mode processor.
|
|
processors = processors.toList();
|
|
for (var processor in processors) {
|
|
if (processor.appliesTo(error)) {
|
|
return processor;
|
|
}
|
|
}
|
|
return null;
|
|
}
|
|
}
|