Fix to suppress (redundant) parse error reporting.

Background: https://github.com/dart-lang/sdk/issues/24911

BUG=24911
R=scheglov@google.com

Review URL: https://codereview.chromium.org/1439403002 .
This commit is contained in:
pq
2015-11-13 10:25:29 -08:00
parent d3abcbaae0
commit a8ef8cf84e
@@ -470,16 +470,8 @@ class ContextManagerImpl implements ContextManager {
Map<String, YamlNode> options;
try {
options = analysisOptionsProvider.getOptions(folder);
} catch (e, stacktrace) {
AnalysisEngine.instance.logger.logError(
'Error processing .analysis_options',
new CaughtException(e, stacktrace));
// TODO(pquitslund): contribute plugin that sends error notification on
// options file.
// Related test:
// context_manager_test.test_analysis_options_parse_failure()
// AnalysisEngine.instance.optionsPlugin.optionsProcessors
// .forEach((OptionsProcessor p) => p.onError(e));
} catch (_) {
// Parse errors are reported by GenerateOptionsErrorsTask.
}
if (options == null && !optionsRemoved) {
@@ -838,7 +830,7 @@ class ContextManagerImpl implements ContextManager {
return new CustomPackageResolverDisposition(resolver);
}
}
ServerPerformanceStatistics.pub.makeCurrentWhile(() {
packageMapInfo = _packageMapProvider.computePackageMap(folder);
});