ListPieces refactor prep for fallible: use filter_map

NFC

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/daemon/cmdlistener.rs b/daemon/cmdlistener.rs
index ec0c83ce1606f2a7b5e69c2d5a9a517d26142138..19af73d5e2e3722a57c1ec80c1e318284f2687c7 100644 (file)
--- a/daemon/cmdlistener.rs
+++ b/daemon/cmdlistener.rs
@@ -375,7 +375,7 @@ fn execute_game_insn<'cs, 'igr, 'ig: 'igr>(
     },
 
     MGI::ListPieces => readonly(cs,ag,ig, &[TP::ViewNotSecret], |ig|{
-      let pieces = ig.gs.pieces.iter().map(|(piece,p)|{
+      let pieces = ig.gs.pieces.iter().filter_map(|(piece,p)| {
         let &PieceState { pos, face, .. } = p;
         let pinfo = ig.ipieces.get(piece)?;
         let desc_html = pinfo.describe_html_infallible(None, p);
@@ -393,7 +393,7 @@ fn execute_game_insn<'cs, 'igr, 'ig: 'igr>(
           piece, itemname,
           visible
         })
-      }).flatten().collect();
+      }).collect();
       Ok(MGR::Pieces(pieces))
     })?,