Deal with linter infos

Author: haarts

example/main.dart

@@ -11,20 +11,20 @@ var loggerNoStack = Logger(
 
 void main() {
   print(
-      "Run with either `dart example/lib/main.dart` or `dart --enable-asserts example/lib/main.dart`.");
+      'Run with either `dart example/lib/main.dart` or `dart --enable-asserts example/lib/main.dart`.');
   demo();
 }
 
 void demo() {
-  logger.d("Log message with 2 methods");
+  logger.d('Log message with 2 methods');
 
-  loggerNoStack.i("Info message");
+  loggerNoStack.i('Info message');
 
-  loggerNoStack.w("Just a warning!");
+  loggerNoStack.w('Just a warning!');
 
-  logger.e("Error! Something bad happened", "Test Error");
+  logger.e('Error! Something bad happened', 'Test Error');
 
-  loggerNoStack.v({"key": 5, "value": "something"});
+  loggerNoStack.v({'key': 5, 'value': 'something'});
 
-  Logger(printer: SimplePrinter()..useColor = true).v("boom");
+  Logger(printer: SimplePrinter()..useColor = true).v('boom');
 }

lib/src/ansi_color.dart

@@ -3,7 +3,7 @@ class AnsiColor {
   static const ansiEsc = '\x1B[';
 
   /// Reset all colors and options for current SGRs to terminal defaults.
-  static const ansiDefault = "${ansiEsc}0m";
+  static const ansiDefault = '${ansiEsc}0m';
 
   final int fg;
   final int bg;
@@ -22,19 +22,20 @@ class AnsiColor {
       : fg = null,
         color = true;
 
+  @override
   String toString() {
     if (fg != null) {
-      return "${ansiEsc}38;5;${fg}m";
+      return '${ansiEsc}38;5;${fg}m';
     } else if (bg != null) {
-      return "${ansiEsc}48;5;${bg}m";
+      return '${ansiEsc}48;5;${bg}m';
     } else {
-      return "";
+      return '';
     }
   }
 
   String call(String msg) {
     if (color) {
-      return "${this}$msg$ansiDefault";
+      return '${this}$msg$ansiDefault';
     } else {
       return msg;
     }
@@ -45,10 +46,10 @@ class AnsiColor {
   AnsiColor toBg() => AnsiColor.bg(fg);
 
   /// Defaults the terminal's foreground color without altering the background.
-  String get resetForeground => color ? "${ansiEsc}39m" : "";
+  String get resetForeground => color ? '${ansiEsc}39m' : '';
 
   /// Defaults the terminal's background color without altering the foreground.
-  String get resetBackground => color ? "${ansiEsc}49m" : "";
+  String get resetBackground => color ? '${ansiEsc}49m' : '';
 
   static int grey(double level) => 232 + (level.clamp(0.0, 1.0) * 23).round();
 }

lib/src/logger.dart

@@ -33,10 +33,10 @@ class OutputEvent {
   OutputEvent(this.level, this.lines);
 }
 
-@Deprecated("Use a custom LogFilter instead")
+@Deprecated('Use a custom LogFilter instead')
 typedef LogCallback = void Function(LogEvent event);
 
-@Deprecated("Use a custom LogOutput instead")
+@Deprecated('Use a custom LogOutput instead')
 typedef OutputCallback = void Function(OutputEvent event);
 
 /// Use instances of logger to send log messages to the [LogPrinter].
@@ -109,11 +109,11 @@ class Logger {
   void log(Level level, dynamic message,
       [dynamic error, StackTrace stackTrace]) {
     if (!_active) {
-      throw ArgumentError("Logger has already been closed.");
+      throw ArgumentError('Logger has already been closed.');
     } else if (error != null && error is StackTrace) {
-      throw ArgumentError("Error parameter cannot take a StackTrace!");
+      throw ArgumentError('Error parameter cannot take a StackTrace!');
     } else if (level == Level.nothing) {
-      throw ArgumentError("Log events cannot have Level.nothing");
+      throw ArgumentError('Log events cannot have Level.nothing');
     }
     var logEvent = LogEvent(level, message, error, stackTrace);
     if (_filter.shouldLog(logEvent)) {
@@ -143,29 +143,29 @@ class Logger {
   }
 
   /// Register a [LogCallback] which is called for each new [LogEvent].
-  @Deprecated("Use a custom LogFilter instead")
+  @Deprecated('Use a custom LogFilter instead')
   static void addLogListener(LogCallback callback) {
     _logCallbacks.add(callback);
   }
 
   /// Removes a [LogCallback] which was previously registered.
   ///
   /// Returns wheter the callback was successfully removed.
-  @Deprecated("Use a custom LogFilter instead")
+  @Deprecated('Use a custom LogFilter instead')
   static bool removeLogListener(LogCallback callback) {
     return _logCallbacks.remove(callback);
   }
 
   /// Register an [OutputCallback] which is called for each new [OutputEvent].
-  @Deprecated("Use a custom LogOutput instead")
+  @Deprecated('Use a custom LogOutput instead')
   static void addOutputListener(OutputCallback callback) {
     _outputCallbacks.add(callback);
   }
 
   /// Removes a [OutputCallback] which was previously registered.
   ///
   /// Returns wheter the callback was successfully removed.
-  @Deprecated("Use a custom LogOutput instead")
+  @Deprecated('Use a custom LogOutput instead')
   static void removeOutputListener(OutputCallback callback) {
     _outputCallbacks.remove(callback);
   }

lib/src/printers/logfmt_printer.dart

@@ -17,7 +17,7 @@ class LogfmtPrinter extends LogPrinter {
 
   @override
   List<String> log(LogEvent event) {
-    var output = StringBuffer("level=${levelPrefixes[event.level]}");
+    var output = StringBuffer('level=${levelPrefixes[event.level]}');
     if (event.message is String) {
       output.write(' msg="${event.message}"');
     } else if (event.message is Map) {

lib/src/printers/pretty_printer.dart

@@ -21,8 +21,8 @@ class PrettyPrinter extends LogPrinter {
   static const bottomLeftCorner = '└';
   static const middleCorner = '├';
   static const verticalLine = '│';
-  static const doubleDivider = "─";
-  static const singleDivider = "┄";
+  static const doubleDivider = '─';
+  static const singleDivider = '┄';
 
   static final levelColors = {
     Level.verbose: AnsiColor.fg(AnsiColor.grey(0.5)),
@@ -69,14 +69,14 @@ class PrettyPrinter extends LogPrinter {
 
     var doubleDividerLine = StringBuffer();
     var singleDividerLine = StringBuffer();
-    for (int i = 0; i < lineLength - 1; i++) {
+    for (var i = 0; i < lineLength - 1; i++) {
       doubleDividerLine.write(doubleDivider);
       singleDividerLine.write(singleDivider);
     }
 
-    _topBorder = "$topLeftCorner$doubleDividerLine";
-    _middleBorder = "$middleCorner$singleDividerLine";
-    _bottomBorder = "$bottomLeftCorner$doubleDividerLine";
+    _topBorder = '$topLeftCorner$doubleDividerLine';
+    _middleBorder = '$middleCorner$singleDividerLine';
+    _bottomBorder = '$bottomLeftCorner$doubleDividerLine';
   }
 
   @override
@@ -109,7 +109,7 @@ class PrettyPrinter extends LogPrinter {
   }
 
   String formatStackTrace(StackTrace stackTrace, int methodCount) {
-    var lines = stackTrace.toString().split("\n");
+    var lines = stackTrace.toString().split('\n');
 
     var formatted = <String>[];
     var count = 0;
@@ -119,7 +119,7 @@ class PrettyPrinter extends LogPrinter {
         if (match.group(2).startsWith('package:logger')) {
           continue;
         }
-        var newLine = "#$count   ${match.group(1)} (${match.group(2)})";
+        var newLine = '#$count   ${match.group(1)} (${match.group(2)})';
         formatted.add(newLine.replaceAll('<anonymous closure>', '()'));
         if (++count == methodCount) {
           break;
@@ -138,23 +138,23 @@ class PrettyPrinter extends LogPrinter {
 
   String getTime() {
     String _threeDigits(int n) {
-      if (n >= 100) return "$n";
-      if (n >= 10) return "0$n";
-      return "00$n";
+      if (n >= 100) return '$n';
+      if (n >= 10) return '0$n';
+      return '00$n';
     }
 
     String _twoDigits(int n) {
-      if (n >= 10) return "$n";
-      return "0$n";
+      if (n >= 10) return '$n';
+      return '0$n';
     }
 
     var now = DateTime.now();
-    String h = _twoDigits(now.hour);
-    String min = _twoDigits(now.minute);
-    String sec = _twoDigits(now.second);
-    String ms = _threeDigits(now.millisecond);
+    var h = _twoDigits(now.hour);
+    var min = _twoDigits(now.minute);
+    var sec = _twoDigits(now.second);
+    var ms = _threeDigits(now.millisecond);
     var timeSinceStart = now.difference(_startTime).toString();
-    return "$h:$min:$sec.$ms (+$timeSinceStart)";
+    return '$h:$min:$sec.$ms (+$timeSinceStart)';
   }
 
   String stringifyMessage(dynamic message) {
@@ -190,7 +190,7 @@ class PrettyPrinter extends LogPrinter {
     if (printEmojis) {
       return levelEmojis[level];
     } else {
-      return "";
+      return '';
     }
   }
 
@@ -201,6 +201,8 @@ class PrettyPrinter extends LogPrinter {
     String error,
     String stacktrace,
   ) {
+    // This code is non trivial and a type annotation here helps understanding.
+    // ignore: omit_local_variable_types
     List<String> buffer = [];
     var color = _getLevelColor(level);
     buffer.add(color(_topBorder));

lib/src/printers/simple_printer.dart

@@ -43,9 +43,9 @@ class SimplePrinter extends LogPrinter {
   @override
   List<String> log(LogEvent event) {
     var messageStr = _stringifyMessage(event.message);
-    var errorStr = event.error != null ? "  ERROR: ${event.error}" : "";
-    var timeStr = printTime ? "TIME: ${DateTime.now().toIso8601String()}" : "";
-    return ["${_labelFor(event.level)} $timeStr $messageStr$errorStr"];
+    var errorStr = event.error != null ? '  ERROR: ${event.error}' : '';
+    var timeStr = printTime ? 'TIME: ${DateTime.now().toIso8601String()}' : '';
+    return ['${_labelFor(event.level)} $timeStr $messageStr$errorStr'];
   }
 
   String _labelFor(Level level) {