No need to convert to array

Jack251970 · Jack251970 · commit 69dd0383115e · 2025-06-05T10:16:02.000+08:00
diff --git a/Plugins/Flow.Launcher.Plugin.BrowserBookmark/ChromiumBookmarkLoader.cs b/Plugins/Flow.Launcher.Plugin.BrowserBookmark/ChromiumBookmarkLoader.cs
@@ -161,7 +161,7 @@ private void LoadFaviconsFromDb(string dbPath, List<Bookmark> bookmarks)
             var savedPaths = new ConcurrentDictionary<string, bool>();
 
             // Get favicons based on bookmarks concurrently
-            Parallel.ForEach(bookmarks.ToArray(), bookmark =>
+            Parallel.ForEach(bookmarks, bookmark =>
             {
                 // Use read-only connection to avoid locking issues
                 var connection = new SqliteConnection($"Data Source={tempDbPath};Mode=ReadOnly");

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,7 @@ private void LoadFaviconsFromDb(string dbPath, List<Bookmark> bookmarks)`
`161`	`161`	`var savedPaths = new ConcurrentDictionary<string, bool>();`
`162`	`162`
`163`	`163`	`// Get favicons based on bookmarks concurrently`
`164`		`- Parallel.ForEach(bookmarks.ToArray(), bookmark =>`
	`164`	`+ Parallel.ForEach(bookmarks, bookmark =>`
`165`	`165`	`{`
`166`	`166`	`// Use read-only connection to avoid locking issues`
`167`	`167`	`var connection = new SqliteConnection($"Data Source={tempDbPath};Mode=ReadOnly");`