Skip to content

Commit 9a86ae5

Browse files
srawlinsCommit Queue
authored andcommitted
DAS: PluginCodeActionsProducer.driver can be private, benefit from promotion
Change-Id: I79d37becada793349f7330de44f1fdb19408b312 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/450141 Reviewed-by: Brian Wilkerson <[email protected]> Commit-Queue: Brian Wilkerson <[email protected]> Auto-Submit: Samuel Rawlins <[email protected]>
1 parent 7cf3727 commit 9a86ae5

File tree

1 file changed

+8
-6
lines changed
  • pkg/analysis_server/lib/src/lsp/handlers/code_actions

1 file changed

+8
-6
lines changed

pkg/analysis_server/lib/src/lsp/handlers/code_actions/plugins.dart

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import 'package:collection/collection.dart';
1616

1717
/// Produces [CodeActionLiteral]s from Plugin fixes and assists.
1818
class PluginCodeActionsProducer extends AbstractCodeActionsProducer {
19-
final AnalysisDriver? driver;
19+
final AnalysisDriver? _driver;
2020

2121
PluginCodeActionsProducer(
2222
super.server,
@@ -31,7 +31,7 @@ class PluginCodeActionsProducer extends AbstractCodeActionsProducer {
3131
required super.allowCommands,
3232
required super.analysisOptions,
3333
required super.allowSnippets,
34-
}) : driver = server.getAnalysisDriver(file.path);
34+
}) : _driver = server.getAnalysisDriver(file.path);
3535

3636
@override
3737
String get name => 'PluginActionsComputer';
@@ -149,12 +149,14 @@ class PluginCodeActionsProducer extends AbstractCodeActionsProducer {
149149
Future<List<plugin.Response>> _sendPluginRequest(
150150
plugin.RequestParams requestParams,
151151
) async {
152-
var driver = this.driver;
153-
if (driver == null) {
154-
return [];
152+
if (_driver == null) {
153+
return const [];
155154
}
156155

157-
var pluginFutures = server.broadcastRequestToPlugins(requestParams, driver);
156+
var pluginFutures = server.broadcastRequestToPlugins(
157+
requestParams,
158+
_driver,
159+
);
158160

159161
return waitForResponses(pluginFutures, requestParameters: requestParams);
160162
}

0 commit comments

Comments
 (0)