Skip to content

Commit 946327f

Browse files
authored
Fixes future warning for awaiting Future returns in async bodies inside try blocks (#9664)
* fixes future warning for `await`ing `Future` returns in `async` bodies inside `try` blocks * fixes new diagnostic
1 parent cf306e3 commit 946327f

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

packages/devtools_app_shared/lib/src/service/eval_on_dart_library.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ class EvalOnDartLibrary extends DisposableController
211211
if (result is ErrorRef) {
212212
throw result;
213213
}
214-
return result as FutureOr<InstanceRef?>;
214+
return await (result as FutureOr<InstanceRef?>);
215215
} catch (e, stack) {
216216
if (shouldLogError) {
217217
_handleError('$e - $expression', stack);
@@ -242,7 +242,7 @@ class EvalOnDartLibrary extends DisposableController
242242
if (result is ErrorRef) {
243243
throw result;
244244
}
245-
return result as FutureOr<InstanceRef?>;
245+
return await (result as FutureOr<InstanceRef?>);
246246
} catch (e, stack) {
247247
if (shouldLogError) {
248248
_handleError('$e - $name', stack);

packages/devtools_shared/test/helpers/helpers.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ Future<TestDtdConnectionInfo> startDtd() async {
5555
}
5656
});
5757

58-
return completer.future
58+
return await completer.future
5959
.timeout(dtdConnectTimeout, onTimeout: onFailure)
6060
.then((value) async {
6161
await dtdStoutSubscription?.cancel();

0 commit comments

Comments
 (0)