refactor: add a type alias for generation ids

This will make it harder to accidentally use a string. Can still be
confused with a chunk id.

Sponsored-by: author

1 files changed, 5 insertions, 3 deletions

diff --git a/src/backup_run.rs b/src/backup_run.rs
index dee1d11..3b107bb 100644
--- a/src/backup_run.rs
+++ b/src/backup_run.rs
@@ -6,7 +6,9 @@ use crate::config::ClientConfig;
 use crate::error::ObnamError;
 use crate::fsentry::FilesystemEntry;
 use crate::fsiter::{AnnotatedFsEntry, FsIterError, FsIterator};
-use crate::generation::{LocalGeneration, LocalGenerationError, NascentError, NascentGeneration};
+use crate::generation::{
+    GenId, LocalGeneration, LocalGenerationError, NascentError, NascentGeneration,
+};
 use crate::policy::BackupPolicy;
 use log::{info, warn};
 use std::path::{Path, PathBuf};
@@ -72,7 +74,7 @@ impl<'a> BackupRun<'a> {
 
     pub fn start(
         &mut self,
-        genid: Option<&str>,
+        genid: Option<&GenId>,
         oldname: &Path,
     ) -> Result<LocalGeneration, ObnamError> {
         match genid {
@@ -94,7 +96,7 @@ impl<'a> BackupRun<'a> {
 
     fn fetch_previous_generation(
         &self,
-        genid: &str,
+        genid: &GenId,
         oldname: &Path,
     ) -> Result<LocalGeneration, ObnamError> {
         let progress = BackupProgress::download_generation(genid);