summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLars Wirzenius <liw@liw.fi>2021-02-07 17:59:53 +0200
committerLars Wirzenius <liw@liw.fi>2021-02-07 18:42:52 +0200
commit5b863edb32314808b6823f90c03ad97da6fdbcbf (patch)
treed85c9fd74f19ecc150e10edd38f7cc1222a0d1d7
parent73a9ca3d90ac393b7b8575199a3fcce855374136 (diff)
downloadobnam2-5b863edb32314808b6823f90c03ad97da6fdbcbf.tar.gz
refactor: rename Reason::Error to Reason::GenerationLookupError
New name is more precise. The meaning of the enum variant hasn't changed.
-rw-r--r--src/backup_reason.rs6
-rw-r--r--src/backup_run.rs5
-rw-r--r--src/policy.rs2
3 files changed, 8 insertions, 5 deletions
diff --git a/src/backup_reason.rs b/src/backup_reason.rs
index 502b79e..b3ce5e9 100644
--- a/src/backup_reason.rs
+++ b/src/backup_reason.rs
@@ -8,7 +8,7 @@ pub enum Reason {
IsNew,
Changed,
Unchanged,
- Error,
+ GenerationLookupError,
Unknown,
}
@@ -19,7 +19,7 @@ impl Reason {
"new" => Reason::IsNew,
"changed" => Reason::Changed,
"unchanged" => Reason::Unchanged,
- "error" => Reason::Error,
+ "genlookuperror" => Reason::GenerationLookupError,
_ => Reason::Unknown,
}
}
@@ -41,7 +41,7 @@ impl fmt::Display for Reason {
Reason::IsNew => "new",
Reason::Changed => "changed",
Reason::Unchanged => "unchanged",
- Reason::Error => "error",
+ Reason::GenerationLookupError => "genlookuperror",
Reason::Unknown => "unknown",
};
write!(f, "{}", reason)
diff --git a/src/backup_run.rs b/src/backup_run.rs
index f97bb58..e0ed533 100644
--- a/src/backup_run.rs
+++ b/src/backup_run.rs
@@ -85,7 +85,10 @@ impl BackupRun {
self.progress.found_live_file(path);
let reason = self.policy.needs_backup(&old, &entry);
match reason {
- Reason::IsNew | Reason::Changed | Reason::Error | Reason::Unknown => {
+ Reason::IsNew
+ | Reason::Changed
+ | Reason::GenerationLookupError
+ | Reason::Unknown => {
let ids = self
.client
.upload_filesystem_entry(&entry, self.buffer_size)?;
diff --git a/src/policy.rs b/src/policy.rs
index 032b851..8a65e09 100644
--- a/src/policy.rs
+++ b/src/policy.rs
@@ -42,7 +42,7 @@ impl BackupPolicy {
"needs_backup: lookup in old generation returned error, ignored: {:?}: {}",
new_name, err
);
- Reason::Error
+ Reason::GenerationLookupError
}
};
debug!(