From 1c5ab8bf1acf8347d39da13c2a9d59c9ba003f3c Mon Sep 17 00:00:00 2001
From: Fallenhh <fallen2hh@gmail.com>
Date: Fri, 19 Jun 2020 23:03:33 +0800
Subject: [PATCH] refactor(cli/checksum): use map to generate hex string
 (#6382)

---
 cli/checksum.rs | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/cli/checksum.rs b/cli/checksum.rs
index 8696f93f7c..ca7590d8ca 100644
--- a/cli/checksum.rs
+++ b/cli/checksum.rs
@@ -1,15 +1,13 @@
-use std::fmt::Write;
-
 pub fn gen(v: Vec<&[u8]>) -> String {
   let mut ctx = ring::digest::Context::new(&ring::digest::SHA256);
   for src in v.iter() {
     ctx.update(src);
   }
   let digest = ctx.finish();
-  let mut out = String::new();
-  // TODO There must be a better way to do this...
-  for byte in digest.as_ref() {
-    write!(&mut out, "{:02x}", byte).unwrap();
-  }
-  out
+  let out: Vec<String> = digest
+    .as_ref()
+    .iter()
+    .map(|byte| format!("{:02x}", byte))
+    .collect();
+  out.join("")
 }