From: Scott Worley <scottworley@scottworley.com>
Date: Sun, 1 Jan 2023 09:38:18 +0000 (-0800)
Subject: Appease clippy more
X-Git-Tag: v0.3.0~6
X-Git-Url: http://git.scottworley.com/voter/commitdiff_plain/971d76c762faaa77ac019efaba5f68478bd82e30?ds=sidebyside

Appease clippy more
---

diff --git a/src/main.rs b/src/main.rs
index ef09e31..fcb16c7 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -51,9 +51,9 @@ fn get_voter(request: &cgi::Request) -> Result<&[u8], cgi::Response> {
     }
 }
 
-fn tally_votes(dir: PathBuf) -> std::io::Result<HashMap<String, HashSet<String>>> {
+fn tally_votes(dir: &Path) -> std::io::Result<HashMap<String, HashSet<String>>> {
     let mut tally: HashMap<String, HashSet<String>> = HashMap::new();
-    if let Ok(vfile) = std::fs::File::open(dir.join("votes")) {
+    if let Ok(vfile) = std::fs::File::open(dir.to_owned().join("votes")) {
         for liner in std::io::BufReader::new(vfile).lines() {
             let line = liner?;
             if let Some((voter, datum)) = line.split_once(' ') {
@@ -77,15 +77,15 @@ fn tally_votes(dir: PathBuf) -> std::io::Result<HashMap<String, HashSet<String>>
     Ok(tally)
 }
 
-fn valid_id_char(c: &u8) -> bool {
-    (b'A'..=b'Z').contains(c) || (b'a'..=b'z').contains(c) || (b'0'..=b'9').contains(c)
+fn valid_id_char(c: u8) -> bool {
+    (b'A'..=b'Z').contains(&c) || (b'a'..=b'z').contains(&c) || (b'0'..=b'9').contains(&c)
 }
 
 fn make_random_id() -> [u8; COOKIE_LENGTH] {
     let mut id = [0; COOKIE_LENGTH];
     for c in &mut id {
-        while !valid_id_char(c) {
-            *c = random()
+        while !valid_id_char(*c) {
+            *c = random();
         }
     }
     id
@@ -195,15 +195,14 @@ fn supports(tally: &HashMap<String, HashSet<String>>, me: &str, candidate: &str)
         .map_or(false, |supporters| supporters.contains(me))
 }
 
-fn prompt_for_vote(dir: PathBuf, request: cgi::Request) -> Result<cgi::Response, cgi::Response> {
-    let voter = get_voter(&request);
+fn prompt_for_vote(dir: &Path, request: &cgi::Request) -> Result<cgi::Response, cgi::Response> {
+    let voter = get_voter(request);
     let me = if let Ok(id) = voter {
         std::str::from_utf8(id).ok()
     } else {
         None
     };
-    let tally =
-        tally_votes(dir.clone()).map_err(|_| cgi::text_response(503, "Couldn't tally votes"))?;
+    let tally = tally_votes(dir).map_err(|_| cgi::text_response(503, "Couldn't tally votes"))?;
     let cfile = std::fs::File::open(dir.join("candidates"))
         .map_err(|_| cgi::text_response(503, "No candidates"))?;
     let mut response = cgi::html_response(
@@ -231,12 +230,12 @@ fn prompt_for_vote(dir: PathBuf, request: cgi::Request) -> Result<cgi::Response,
             .map_err(|_| cgi::text_response(503, "Missing candidates"))?,
     );
     if voter.is_err() {
-        response = set_cookie(response, request.uri().path())?
+        response = set_cookie(response, request.uri().path())?;
     }
     Ok(response)
 }
 
-fn write_vote(dir: PathBuf, voter: &[u8], vote: &[u8]) -> std::io::Result<()> {
+fn write_vote(dir: &Path, voter: &[u8], vote: &[u8]) -> std::io::Result<()> {
     let datum = [voter, b" ", vote, b"\n"].concat();
     let vpath = dir.join("votes");
     let vfile = std::fs::File::options()
@@ -248,7 +247,7 @@ fn write_vote(dir: PathBuf, voter: &[u8], vote: &[u8]) -> std::io::Result<()> {
     Ok(())
 }
 
-fn record_vote(dir: PathBuf, request: cgi::Request) -> Result<cgi::Response, cgi::Response> {
+fn record_vote(dir: &Path, request: &cgi::Request) -> Result<cgi::Response, cgi::Response> {
     let body = request.body();
     // Valid votes look like "0 foo" or "1 bar"
     if body.len() < 3
@@ -258,7 +257,7 @@ fn record_vote(dir: PathBuf, request: cgi::Request) -> Result<cgi::Response, cgi
     {
         return Err(cgi::text_response(415, "Invalid vote"));
     }
-    write_vote(dir, get_voter(&request)?, body)
+    write_vote(dir, get_voter(request)?, body)
         .map_err(|_| cgi::text_response(503, "Couldn't record vote"))?;
     Ok(cgi::text_response(200, "Vote recorded"))
 }
@@ -268,18 +267,18 @@ fn strip_body(mut response: cgi::Response) -> cgi::Response {
     response
 }
 
-fn respond(request: cgi::Request) -> Result<cgi::Response, cgi::Response> {
+fn respond(request: &cgi::Request) -> Result<cgi::Response, cgi::Response> {
     let dir = validate_path(request.uri().path())?;
     match *request.method() {
-        cgi::http::Method::HEAD => prompt_for_vote(dir, request).map(strip_body),
-        cgi::http::Method::GET => prompt_for_vote(dir, request),
-        cgi::http::Method::PUT => record_vote(dir, request),
+        cgi::http::Method::HEAD => prompt_for_vote(&dir, request).map(strip_body),
+        cgi::http::Method::GET => prompt_for_vote(&dir, request),
+        cgi::http::Method::PUT => record_vote(&dir, request),
         _ => Err(cgi::text_response(405, "Huh?")),
     }
 }
 
 fn respond_or_report_error(request: cgi::Request) -> cgi::Response {
-    match respond(request) {
+    match respond(&request) {
         Ok(result) => result,
         Err(error) => error,
     }