chore: improve code style

lib/src/imap/imap_client.dart

@@ -2246,6 +2246,7 @@ class ImapClient extends ClientBase {
       responseTimeout: defaultResponseTimeout,
     );
     final result = await sendCommand<Mailbox?>(cmd, NoopParser(this, box));
+
     return result ?? box;
   }
 

lib/src/mail/mail_exception.dart

@@ -6,21 +6,37 @@ class MailException implements Exception {
   MailException(this.mailClient, this.message, {this.stackTrace, this.details});
 
   /// Creates a new exception from the low level one
-  MailException.fromImap(MailClient mailClient, ImapException e,
-      [StackTrace? s])
-      : this(mailClient, '${e.imapClient.logName}:  ${e.message}',
-            stackTrace: s ?? e.stackTrace, details: e.details);
+  MailException.fromImap(
+    MailClient mailClient,
+    ImapException e, [
+    StackTrace? s,
+  ]) : this(
+          mailClient,
+          '${e.imapClient.logName}:  ${e.message}',
+          stackTrace: s ?? e.stackTrace,
+          details: e.details,
+        );
 
   /// Creates a new exception from the low level one
   MailException.fromPop(MailClient mailClient, PopException e, [StackTrace? s])
-      : this(mailClient, '${e.popClient.logName}:  ${e.message}',
-            stackTrace: s ?? e.stackTrace, details: e.response);
+      : this(
+          mailClient,
+          '${e.popClient.logName}:  ${e.message}',
+          stackTrace: s ?? e.stackTrace,
+          details: e.response,
+        );
 
   /// Creates a new exception from the low level one
-  MailException.fromSmtp(MailClient mailClient, SmtpException e,
-      [StackTrace? s])
-      : this(mailClient, '${e.smtpClient.logName}:  ${e.message}',
-            stackTrace: s ?? e.stackTrace, details: e.response);
+  MailException.fromSmtp(
+    MailClient mailClient,
+    SmtpException e, [
+    StackTrace? s,
+  ]) : this(
+          mailClient,
+          '${e.smtpClient.logName}:  ${e.message}',
+          stackTrace: s ?? e.stackTrace,
+          details: e.response,
+        );
 
   /// The originating mail client
   final MailClient mailClient;
@@ -49,6 +65,7 @@ class MailException implements Exception {
         ..write('\n')
         ..write(stackTrace);
     }
+
     return buffer.toString();
   }
 }

lib/src/mail_conventions.dart

@@ -68,7 +68,7 @@ This is no guarantee that the message has been read or understood.''';
     'RES', // Portuguese
     'Odp', // Polnish
     'YNT', // Turkish
-    'ATB' // Welsh
+    'ATB', // Welsh
   ];
 
   /// Common abbreviations in subject header for forwarded messages,
@@ -97,7 +97,7 @@ This is no guarantee that the message has been read or understood.''';
     'ENC', // Portuguese
     'PD', // Polnish
     'İLT', // Turkish
-    'YML' // Welsh
+    'YML', // Welsh
   ];
   // cSpell:enable
 

lib/src/media_type.dart

@@ -301,9 +301,11 @@ class MediaType {
       final topText = lcText.substring(0, splitPos);
       final top = _topLevelByMimeName[topText] ?? MediaToptype.other;
       final sub = _subtypesByMimeType[lcText] ?? MediaSubtype.other;
+
       return MediaType(lcText, top, sub);
     } else {
       final top = _topLevelByMimeName[lcText] ?? MediaToptype.other;
+
       return MediaType(lcText, top, MediaSubtype.other);
     }
   }
@@ -317,12 +319,14 @@ class MediaType {
         if (splitPos != -1) {
           final topText = key.substring(0, splitPos);
           final top = _topLevelByMimeName[topText] ?? MediaToptype.other;
+
           return MediaType(key, top, subtype);
         }
         break;
       }
     }
     print('Error: unable to resolve media subtype $subtype');
+
     return MediaType('example/example', MediaToptype.other, subtype);
   }
 
@@ -336,8 +340,10 @@ class MediaType {
     final lastDotIndex = fileNameOrPath.lastIndexOf('.');
     if (lastDotIndex != -1 && lastDotIndex < fileNameOrPath.length - 1) {
       final ext = fileNameOrPath.substring(lastDotIndex + 1).toLowerCase();
+
       return MediaType.guessFromFileExtension(ext);
     }
+
     return MediaSubtype.applicationOctetStream.mediaType;
   }
 
@@ -382,6 +388,7 @@ class MediaType {
       case 'zip':
         return MediaSubtype.applicationZip.mediaType;
     }
+
     return MediaSubtype.applicationOctetStream.mediaType;
   }
   // cSpell:enable
@@ -492,7 +499,7 @@ class MediaType {
     'font/ttf': MediaSubtype.fontTtf,
     'font/woff': MediaSubtype.fontWoff,
     'font/woff2': MediaSubtype.fontWoff2,
-    'font/collection': MediaSubtype.fontCollection
+    'font/collection': MediaSubtype.fontCollection,
   };
   // cSpell:enable
 

lib/src/pop/pop_client.dart

@@ -70,7 +70,9 @@ class PopClient extends ClientBase {
 
   @override
   FutureOr<void> onConnectionEstablished(
-      ConnectionInfo connectionInfo, String serverGreeting) {
+    ConnectionInfo connectionInfo,
+    String serverGreeting,
+  ) {
     if (serverGreeting.startsWith('+OK')) {
       final chunks = serverGreeting.split(' ');
       serverInfo = PopServerInfo(chunks.last.trimRight());
@@ -91,6 +93,7 @@ class PopClient extends ClientBase {
     final currentLine = _currentFirstResponseLine;
     if (currentLine != null && currentLine.startsWith('-ERR')) {
       onServerResponse([currentLine]);
+
       return;
     }
     if (_currentCommand?.isMultiLine ?? false) {
@@ -179,6 +182,7 @@ class PopClient extends ClientBase {
     _currentCommand = command;
     _currentFirstResponseLine = null;
     writeText(command.command, command);
+
     return command.completer.future;
   }
 

lib/src/pop/pop_exception.dart

@@ -40,6 +40,7 @@ class PopException implements Exception {
         ..write('\n')
         ..write(stackTrace);
     }
+
     return buffer.toString();
   }
 }

lib/src/private/imap/command.dart

@@ -58,6 +58,7 @@ class Command {
     }
     final nextPart = parts[_currentPartIndex];
     _currentPartIndex++;
+
     return nextPart;
   }
 }

lib/src/private/imap/enable_parser.dart

@@ -13,21 +13,28 @@ class EnableParser extends ResponseParser<List<Capability>> {
 
   @override
   List<Capability>? parse(
-      ImapResponse imapResponse, Response<List<Capability>> response) {
+    ImapResponse imapResponse,
+    Response<List<Capability>> response,
+  ) {
     if (response.isOkStatus) {
       return info.enabledCapabilities;
     }
+
     return null;
   }
 
   @override
   bool parseUntagged(
-      ImapResponse imapResponse, Response<List<Capability>>? response) {
+    ImapResponse imapResponse,
+    Response<List<Capability>>? response,
+  ) {
     final line = imapResponse.parseText;
     if (line.startsWith('ENABLED ')) {
       parseCapabilities(line, 'ENABLED '.length);
+
       return true;
     }
+
     return super.parseUntagged(imapResponse, response);
   }
 

lib/src/private/imap/generic_parser.dart

@@ -23,7 +23,9 @@ class GenericParser extends ResponseParser<GenericImapResult> {
   final GenericImapResult _result = GenericImapResult();
   @override
   GenericImapResult parse(
-      ImapResponse imapResponse, Response<GenericImapResult> response) {
+    ImapResponse imapResponse,
+    Response<GenericImapResult> response,
+  ) {
     final text = imapResponse.parseText;
     final startIndex = text.indexOf('[');
     if (startIndex != -1 && startIndex < text.length - 2) {
@@ -35,6 +37,7 @@ class GenericParser extends ResponseParser<GenericImapResult> {
       }
     }
     _result.details ??= text;
+
     return _result;
   }
 
@@ -46,15 +49,18 @@ class GenericParser extends ResponseParser<GenericImapResult> {
     final text = imapResponse.parseText;
     if (text.startsWith('NO ')) {
       _result.warnings.add(ImapWarning('NO', text.substring('NO '.length)));
+
       return true;
     } else if (text.startsWith('BAD ')) {
       _result.warnings.add(ImapWarning('BAD', text.substring('BAD '.length)));
+
       return true;
     } else if (text.startsWith('OK [COPYUID')) {
       final endIndex = text.lastIndexOf(']');
       if (endIndex != -1) {
         _result.responseCode = text.substring('OK ['.length, endIndex);
       }
+
       return true;
     } else if (text.endsWith('EXPUNGE')) {
       // this is the expunge response for a MOVE operation, ignore
@@ -78,6 +84,7 @@ class GenericParser extends ResponseParser<GenericImapResult> {
           ),
         );
       }
+
       return true;
     } else if (text.endsWith('RECENT')) {
       // a message has been added to the current mailbox,
@@ -93,8 +100,10 @@ class GenericParser extends ResponseParser<GenericImapResult> {
           ),
         );
       }
+
       return true;
     }
+
     return super.parseUntagged(imapResponse, response);
   }
 

lib/src/private/imap/id_parser.dart

@@ -12,6 +12,7 @@ class IdParser extends ResponseParser<Id?> {
     if (response.isOkStatus) {
       return _id;
     }
+
     return null;
   }
 
@@ -20,9 +21,11 @@ class IdParser extends ResponseParser<Id?> {
     final text = imapResponse.parseText;
     if (text.startsWith('ID ')) {
       _id = Id.fromText(text.substring('ID '.length));
+
       return true;
     } else if (text.startsWith('* ID ')) {
       _id = Id.fromText(text.substring('* ID '.length));
+
       return true;
     }
 

lib/src/private/imap/imap_response_line.dart

@@ -61,6 +61,7 @@ class ImapResponseLine {
   /// Does this line have a [literal] data indicator?
   bool get isWithLiteral {
     final literal = this.literal;
+
     return literal != null && literal >= 0;
   }
 

lib/src/private/imap/logout_parser.dart

@@ -14,8 +14,10 @@ class LogoutParser extends ResponseParser<String> {
   bool parseUntagged(ImapResponse imapResponse, Response<String>? response) {
     if (imapResponse.parseText.startsWith('BYE')) {
       _bye = imapResponse.parseText;
+
       return true;
     }
+
     return super.parseUntagged(imapResponse, response);
   }
 }

lib/src/private/imap/noop_parser.dart

@@ -30,10 +30,14 @@ class NoopParser extends ResponseParser<Mailbox?> {
       final highestModSequenceIndex =
           imapResponse.parseText.indexOf('[HIGHESTMODSEQ ');
       if (highestModSequenceIndex != -1) {
-        box.highestModSequence = ParserHelper.parseInt(imapResponse.parseText,
-            highestModSequenceIndex + '[HIGHESTMODSEQ '.length, ']');
+        box.highestModSequence = ParserHelper.parseInt(
+          imapResponse.parseText,
+          highestModSequenceIndex + '[HIGHESTMODSEQ '.length,
+          ']',
+        );
       }
     }
+
     return response.isOkStatus ? box : null;
   }
 
@@ -78,6 +82,7 @@ class NoopParser extends ResponseParser<Mailbox?> {
               ),
             );
           }
+
           return true;
         } else {
           if (_fetchParser.parseUntagged(imapResponse, _fetchResponse)) {
@@ -93,6 +98,7 @@ class NoopParser extends ResponseParser<Mailbox?> {
                 ),
               );
             }
+
             return true;
           }
         }
@@ -101,8 +107,10 @@ class NoopParser extends ResponseParser<Mailbox?> {
         // a common response in IDLE mode can be "* OK still here" or similar
         handled = true;
       }
+
       return handled;
     }
+
     return true;
   }