From 52a3a3a3abb9641c33779a3efdd0efa922f6a1e4 Mon Sep 17 00:00:00 2001 From: "thomashii@dukefirehawk.com" Date: Sat, 27 Aug 2022 17:23:41 +0800 Subject: [PATCH] Fixed production analyze warnings --- packages/production/lib/src/runner.dart | 23 ++++++++++------------- packages/proxy/test/pub_serve_test.dart | 7 +++---- 2 files changed, 13 insertions(+), 17 deletions(-) diff --git a/packages/production/lib/src/runner.dart b/packages/production/lib/src/runner.dart index f9dd1d4c..1b70b5b4 100644 --- a/packages/production/lib/src/runner.dart +++ b/packages/production/lib/src/runner.dart @@ -119,9 +119,9 @@ _ ___ | /| / / /_/ / _ /___ _ /___ var onLogRecord = ReceivePort(); var onExit = ReceivePort(); var onError = ReceivePort(); - var runnerArgs = _RunnerArgs(name, configureServer, options, reflector, + var runnerArgs = RunnerArgs(name, configureServer, options, reflector, onLogRecord.sendPort, pubSubSendPort); - var argsWithId = _RunnerArgsWithId(id, runnerArgs); + var argsWithId = RunnerArgsWithId(id, runnerArgs); Isolate.spawn(isolateMain, argsWithId, onExit: onExit.sendPort, @@ -187,11 +187,8 @@ _ ___ | /| / / /_/ / _ /___ _ /___ } } - print(darkGray.wrap(asciiArt + - '\n\n' + - 'A batteries-included, full-featured, full-stack framework in Dart.' + - '\n\n' + - 'https://angel3-framework.web.app\n')); + print(darkGray.wrap( + '$asciiArt\n\nA batteries-included, full-featured, full-stack framework in Dart.\n\nhttps://angel3-framework.web.app\n')); if (argResults['help'] == true) { stdout @@ -232,7 +229,7 @@ _ ___ | /| / / /_/ / _ /___ _ /___ } /// Run with main isolate - static void isolateMain(_RunnerArgsWithId argsWithId) { + static void isolateMain(RunnerArgsWithId argsWithId) { var args = argsWithId.args; hierarchicalLoggingEnabled = false; @@ -304,14 +301,14 @@ _ ___ | /| / / /_/ / _ /___ _ /___ } } -class _RunnerArgsWithId { +class RunnerArgsWithId { final int id; - final _RunnerArgs args; + final RunnerArgs args; - _RunnerArgsWithId(this.id, this.args); + RunnerArgsWithId(this.id, this.args); } -class _RunnerArgs { +class RunnerArgs { final String name; final AngelConfigurer configureServer; @@ -322,7 +319,7 @@ class _RunnerArgs { final SendPort loggingSendPort, pubSubSendPort; - _RunnerArgs(this.name, this.configureServer, this.options, this.reflector, + RunnerArgs(this.name, this.configureServer, this.options, this.reflector, this.loggingSendPort, this.pubSubSendPort); String get loggerName => name; diff --git a/packages/proxy/test/pub_serve_test.dart b/packages/proxy/test/pub_serve_test.dart index 33f473f5..c1b01880 100644 --- a/packages/proxy/test/pub_serve_test.dart +++ b/packages/proxy/test/pub_serve_test.dart @@ -22,7 +22,7 @@ void main() { testApp!.get('/empty', (req, res) => res.close()); testApp!.responseFinalizers.add((req, res) async { - print('OUTGOING: ' + String.fromCharCodes(res.buffer!.toBytes())); + print('OUTGOING: ${String.fromCharCodes(res.buffer!.toBytes())}'); }); testApp!.encoders.addAll({'gzip': gzip.encoder}); @@ -44,9 +44,8 @@ void main() { app!.fallback(layer.handleRequest); app!.responseFinalizers.add((req, res) async { - print('Normal. Buf: ' + - String.fromCharCodes(res.buffer!.toBytes()) + - ', headers: ${res.headers}'); + print( + 'Normal. Buf: ${String.fromCharCodes(res.buffer!.toBytes())}, headers: ${res.headers}'); }); app!.encoders.addAll({'gzip': gzip.encoder});