Replaced Box<dyn Iterator... with impl Iterator

src/fs/explorer/mod.rs

@@ -137,7 +137,7 @@ impl FileExplorer {
     ///
     /// Iterate over files
     /// Filters are applied based on current options (e.g. hidden files not returned)
-    pub fn iter_files(&self) -> Box<dyn Iterator<Item = &FsEntry> + '_> {
+    pub fn iter_files(&self) -> impl Iterator<Item = &FsEntry> + '_ {
         // Filter
         let opts: ExplorerOpts = self.opts;
         Box::new(self.files.iter().filter(move |x| {
@@ -154,7 +154,7 @@ impl FileExplorer {
     /// ### iter_files_all
     ///
     /// Iterate all files; doesn't care about options
-    pub fn iter_files_all(&self) -> Box<dyn Iterator<Item = &FsEntry> + '_> {
+    pub fn iter_files_all(&self) -> impl Iterator<Item = &FsEntry> + '_ {
         Box::new(self.files.iter())
     }
 

src/system/bookmarks_client.rs

@@ -96,7 +96,7 @@ impl BookmarksClient {
     /// ### iter_bookmarks
     ///
     /// Iterate over bookmarks keys
-    pub fn iter_bookmarks(&self) -> Box<dyn Iterator<Item = &String> + '_> {
+    pub fn iter_bookmarks(&self) -> impl Iterator<Item = &String> + '_ {
         Box::new(self.hosts.bookmarks.keys())
     }
 
@@ -156,7 +156,7 @@ impl BookmarksClient {
     /// ### iter_recents
     ///
     /// Iterate over recents keys
-    pub fn iter_recents(&self) -> Box<dyn Iterator<Item = &String> + '_> {
+    pub fn iter_recents(&self) -> impl Iterator<Item = &String> + '_ {
         Box::new(self.hosts.recents.keys())
     }
 

src/system/config_client.rs

@@ -242,7 +242,7 @@ impl ConfigClient {
     /// ### iter_ssh_keys
     ///
     /// Get an iterator through hosts in the ssh key storage
-    pub fn iter_ssh_keys(&self) -> Box<dyn Iterator<Item = &String> + '_> {
+    pub fn iter_ssh_keys(&self) -> impl Iterator<Item = &String> + '_ {
         Box::new(self.config.remote.ssh_keys.keys())
     }