Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(150)

Side by Side Diff: pkg/analyzer_plugin/lib/plugin/plugin.dart

Issue 2688253002: Suppress a few analysis warnings in the analyzer codebase. (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import 'package:analyzer/file_system/file_system.dart'; 5 import 'package:analyzer/file_system/file_system.dart';
6 import 'package:analyzer/src/dart/analysis/byte_store.dart'; 6 import 'package:analyzer/src/dart/analysis/byte_store.dart';
7 import 'package:analyzer/src/dart/analysis/driver.dart' 7 import 'package:analyzer/src/dart/analysis/driver.dart'
8 show AnalysisDriverScheduler, PerformanceLog; 8 show AnalysisDriverScheduler, PerformanceLog;
9 import 'package:analyzer/src/dart/analysis/file_byte_store.dart'; 9 import 'package:analyzer/src/dart/analysis/file_byte_store.dart';
10 import 'package:analyzer/src/dart/analysis/file_state.dart'; 10 import 'package:analyzer/src/dart/analysis/file_state.dart';
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 * The performance log used by any analysis drivers that are created. 56 * The performance log used by any analysis drivers that are created.
57 */ 57 */
58 final PerformanceLog performanceLog = 58 final PerformanceLog performanceLog =
59 new PerformanceLog(new NullStringSink()); 59 new PerformanceLog(new NullStringSink());
60 60
61 /** 61 /**
62 * The byte store used by any analysis drivers that are created, or `null` if 62 * The byte store used by any analysis drivers that are created, or `null` if
63 * the cache location isn't known because the 'plugin.version' request has not 63 * the cache location isn't known because the 'plugin.version' request has not
64 * yet been received. 64 * yet been received.
65 */ 65 */
66 ByteStore _byteStore; 66 ByteStore _byteStore; // ignore: unused_field
scheglov 2017/02/10 21:06:53 I think Brian will use it eventually, and again, I
devoncarew 2017/02/10 21:12:35 Removed the ignore.
67 67
68 /** 68 /**
69 * The file content overlay used by any analysis drivers that are created. 69 * The file content overlay used by any analysis drivers that are created.
70 */ 70 */
71 final FileContentOverlay fileContentOverlay = new FileContentOverlay(); 71 final FileContentOverlay fileContentOverlay = new FileContentOverlay();
72 72
73 /** 73 /**
74 * Initialize a newly created analysis server plugin. If a resource [provider] 74 * Initialize a newly created analysis server plugin. If a resource [provider]
75 * is given, then it will be used to access the file system. Otherwise a 75 * is given, then it will be used to access the file system. Otherwise a
76 * resource provider that accesses the physical file system will be used. 76 * resource provider that accesses the physical file system will be used.
(...skipping 319 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 response = new Response(id, 396 response = new Response(id,
397 error: new RequestError( 397 error: new RequestError(
398 RequestErrorCode.PLUGIN_ERROR, exception.toString(), 398 RequestErrorCode.PLUGIN_ERROR, exception.toString(),
399 stackTrace: stackTrace.toString())); 399 stackTrace: stackTrace.toString()));
400 } 400 }
401 if (response != null) { 401 if (response != null) {
402 _channel.sendResponse(response); 402 _channel.sendResponse(response);
403 } 403 }
404 } 404 }
405 } 405 }
OLDNEW
« pkg/analyzer/test/generated/all_the_rest_test.dart ('K') | « pkg/analyzer/test/source/test_all.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698