From 7996d9a525771363e323d6ffe98f26aa9b6873d0 Mon Sep 17 00:00:00 2001
From: Craig Drummond <craig.p.drummond@gmail.com>
Date: Mon, 7 Mar 2022 19:49:05 +0000
Subject: [PATCH] If fail to remove old tracks from DB, then output more info.

---
 ChangeLog | 1 +
 src/db.rs | 9 ++++++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index a3d6b46..52e47f3 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,6 +4,7 @@
 2. Update user docs.
 3. Update ignore syntax to allow adding SQL WHERE clauses.
 4. Use newer version of tag reader library.
+5. If fail to remove old tracks from DB, then output more info.
 
 0.0.1
 -----
diff --git a/src/db.rs b/src/db.rs
index da0015d..17eb513 100644
--- a/src/db.rs
+++ b/src/db.rs
@@ -169,12 +169,19 @@ impl Db {
         }
         log::info!("Num non-existant tracks: {}", to_remove.len());
         if !dry_run {
+            let count_before = self.get_track_count();
+            let num_to_remove = to_remove.len();
             for t in to_remove {
+                log::debug!("Remove '{}'", t);
                 match self.conn.execute("DELETE FROM Tracks WHERE File = ?;", params![t]) {
                     Ok(_) => { },
-                    Err(_) => { }
+                    Err(e) => { log::error!("Failed to remove '{}' - {}", t, e) }
                 }
             }
+            let count_now = self.get_track_count();
+            if (count_now + num_to_remove) != count_before {
+                log::error!("Failed to remove all tracks. Count before: {}, wanted to remove: {}, count now: {}", count_before, num_to_remove, count_now);
+            }
         }
     }