From ba55d298ceebfc6e661705e8f59a088cc9f7c285 Mon Sep 17 00:00:00 2001
From: "Dustin J. Mitchell" <dustin@mozilla.com>
Date: Mon, 23 Nov 2020 14:29:51 -0500
Subject: [PATCH] stop ignoring dead code and unused variables

---
 taskchampion/src/errors.rs         | 3 ---
 taskchampion/src/lib.rs            | 4 ----
 taskchampion/src/taskstorage/kv.rs | 4 ----
 3 files changed, 11 deletions(-)

diff --git a/taskchampion/src/errors.rs b/taskchampion/src/errors.rs
index 8a180936f..08884376f 100644
--- a/taskchampion/src/errors.rs
+++ b/taskchampion/src/errors.rs
@@ -4,7 +4,4 @@ use failure::Fail;
 pub enum Error {
     #[fail(display = "Task Database Error: {}", _0)]
     DBError(String),
-
-    #[fail(display = "Replica Error: {}", _0)]
-    ReplicaError(String),
 }
diff --git a/taskchampion/src/lib.rs b/taskchampion/src/lib.rs
index c9a242359..9042dc2de 100644
--- a/taskchampion/src/lib.rs
+++ b/taskchampion/src/lib.rs
@@ -1,7 +1,3 @@
-// TODO: remove this eventually when there's an API
-#![allow(dead_code)]
-#![allow(unused_variables)]
-
 #[macro_use]
 extern crate failure;
 
diff --git a/taskchampion/src/taskstorage/kv.rs b/taskchampion/src/taskstorage/kv.rs
index 08f2ade01..da275c90e 100644
--- a/taskchampion/src/taskstorage/kv.rs
+++ b/taskchampion/src/taskstorage/kv.rs
@@ -177,7 +177,6 @@ impl<'t> TaskStorageTxn for Txn<'t> {
     fn all_tasks(&mut self) -> Fallible<Vec<(Uuid, TaskMap)>> {
         let bucket = self.tasks_bucket();
         let kvtxn = self.kvtxn();
-        let curs = kvtxn.read_cursor(bucket)?;
         let all_tasks: Result<Vec<(Uuid, TaskMap)>, Error> = kvtxn
             .read_cursor(bucket)?
             .iter()
@@ -189,7 +188,6 @@ impl<'t> TaskStorageTxn for Txn<'t> {
     fn all_task_uuids(&mut self) -> Fallible<Vec<Uuid>> {
         let bucket = self.tasks_bucket();
         let kvtxn = self.kvtxn();
-        let curs = kvtxn.read_cursor(bucket)?;
         Ok(kvtxn
             .read_cursor(bucket)?
             .iter()
@@ -224,7 +222,6 @@ impl<'t> TaskStorageTxn for Txn<'t> {
     fn operations(&mut self) -> Fallible<Vec<Operation>> {
         let bucket = self.operations_bucket();
         let kvtxn = self.kvtxn();
-        let curs = kvtxn.read_cursor(bucket)?;
         let all_ops: Result<Vec<(u64, Operation)>, Error> = kvtxn
             .read_cursor(bucket)?
             .iter()
@@ -299,7 +296,6 @@ impl<'t> TaskStorageTxn for Txn<'t> {
             res.push(None)
         }
 
-        let curs = kvtxn.read_cursor(working_set_bucket)?;
         for (i, u) in kvtxn.read_cursor(working_set_bucket)?.iter() {
             let i: u64 = i.into();
             res[i as usize] = Some(u.inner()?.to_serde());