minor performance improvement to ParseSummaryItems/Async

jstedfast · Dec 30, 2023 · 46df9e8 · 46df9e8
1 parent adfc55a
commit 46df9e8
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/MailKit/Net/Imap/ImapFolderFetch.cs b/MailKit/Net/Imap/ImapFolderFetch.cs
@@ -317,11 +317,12 @@ void ParseSummaryItems (ImapEngine engine, MessageSummary message, FetchSummaryI
 
 								atom = (string) token.Value;
 
-								headerFields = atom.Equals ("HEADER.FIELDS", StringComparison.OrdinalIgnoreCase);
-
-								if (!headerFields && atom.Equals ("HEADER", StringComparison.OrdinalIgnoreCase)) {
+								if (atom.Equals ("HEADER", StringComparison.OrdinalIgnoreCase)) {
 									// if we're fetching *all* headers, then it will include the References header (if it exists)
 									referencesField = true;
+									headerFields = false;
+								} else {
+									headerFields = atom.Equals ("HEADER.FIELDS", StringComparison.OrdinalIgnoreCase);
 								}
 							}
 						} while (true);
@@ -553,11 +554,12 @@ async Task ParseSummaryItemsAsync (ImapEngine engine, MessageSummary message, Fe
 
 								atom = (string) token.Value;
 
-								headerFields = atom.Equals ("HEADER.FIELDS", StringComparison.OrdinalIgnoreCase);
-
-								if (!headerFields && atom.Equals ("HEADER", StringComparison.OrdinalIgnoreCase)) {
+								if (atom.Equals ("HEADER", StringComparison.OrdinalIgnoreCase)) {
 									// if we're fetching *all* headers, then it will include the References header (if it exists)
 									referencesField = true;
+									headerFields = false;
+								} else {
+									headerFields = atom.Equals ("HEADER.FIELDS", StringComparison.OrdinalIgnoreCase);
 								}
 							}
 						} while (true);