build: bump version (v1.2.2)

Flake input changes:

* Updated 'nixpkgs': 'github:NixOS/nixpkgs/b881f100f65eacbbafdf65cff78c8d4104a1d04a' -> 'github:NixOS/nixpkgs/26f6af373ec1b867d751b56fb802f14010c8351b'

.editorconfig

@@ -1,7 +1,7 @@
 root = true
 
 [*]
-indent_style = space
+indent_style = tab
 indent_size = 4
 charset = utf-8
 trim_trailing_whitespace = true

.rustfmt.toml

@@ -0,0 +1 @@
+hard_tabs = true

CHANGELOG.md

@@ -0,0 +1,19 @@
+# Changelog
+
+## v1.2.2
+
+* dependencies upgraded
+
+## v1.2.1
+
+* dependencies upgraded
+
+## v1.2.0
+
+* "copy" encoding format added
+* "jobs" cli argument added, that lets you set the number of concurrent transcodes
+
+## v1.1.0
+
+* "flac" encoding format added
+* resampling quality set to highest/"10"

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "audio-conv"
-version = "1.0.0"
+version = "1.2.2"
 edition = "2018"
 description = "Copies directory structure and converts audio files in it"
 authors = ["Thomas Heck <t@b128.net>"]
@@ -13,12 +13,18 @@ categories = [
     "multimedia::encoding",
 ]
 keywords = ["audio", "conversion", "opus", "flac"]
+include = [
+    "/src/**/*",
+    "/example.audio-conv.yaml",
+    "/README.md",
+    "/CHANGELOG.md",
+]
 
 [dependencies]
-gstreamer-audio = { version = "0.16", features = ["v1_10"] }
-gstreamer = { version = "0.16", features = ["v1_10"] }
-gstreamer-base = { version = "0.16", features = ["v1_10"] }
-glib = "0.10"
+gstreamer-audio = { version = "0.17", features = ["v1_10"] }
+gstreamer = { version = "0.17", features = ["v1_10"] }
+gstreamer-base = { version = "0.17", features = ["v1_10"] }
+glib = "0.14"
 futures = "0.3"
 num_cpus = "1"
 walkdir = "2"
@@ -30,7 +36,7 @@ serde_yaml = "0.8"
 regex = "1"
 globset = "0.4"
 derive_more = "0.99"
-tui = { version = "0.13", default-features = false, features = ["crossterm"] }
+tui = { version = "0.16", default-features = false, features = ["crossterm"] }
 
 [dependencies.tokio]
 version = "1"

README.md

@@ -0,0 +1,34 @@
+# audio-conv
+
+Takes two paths, all audio files encountered in the first path are transcoded and stored in the
+second path. The directory structure from the first path gets also copied to the second path.
+
+## Dependencies
+
+Requires *gstreamer* version 1.10 or higher with the *base* plugin.
+
+The supported source audio formats (or even other media that is able to contain audio) depend on
+the installed *gstreamer* plugins.
+
+## Installation via nix flakes
+
+*audio-conv* can be easily installed via *nix flakes*:
+
+```bash
+$ nix profile install gitlab:chpio/audio-conv/release
+```
+
+## Generate example config
+
+*audio-conv* is able to write an example config to your current directory:
+
+```bash
+$ audio-conv init
+```
+
+Now you need to edit the generated *audio-conv.yaml* file. And let it convert your audio files
+by running it:
+
+```bash
+$ audio-conv
+```

example.audio-conv.yaml

@@ -13,9 +13,19 @@ matches:
       bitrate: 160
       bitrate_type: vbr # or cbr
 
+    # for copy (copies file without transcoding it):
+    # to:
+    #   codec: copy
+
     # for mp3:
     # to:
     #   codec: mp3
     #   # one of: 8, 16, 24, 32, 40, 48, 56, 64, 80, 96, 112, 128, 160, 192, 224, 256 or 320
     #   bitrate: 256
     #   bitrate_type: vbr # or cbr
+
+    # for flac:
+    # to:
+    #   codec: flac
+    #   # effort spend for the compression. 0 (fastes compression) to 9 (highest compression)
+    #   compression: 8

flake.lock

@@ -2,11 +2,11 @@
   "nodes": {
     "flake-utils": {
       "locked": {
-        "lastModified": 1605370193,
-        "narHash": "sha256-YyMTf3URDL/otKdKgtoMChu4vfVL3vCMkRqpGifhUn0=",
+        "lastModified": 1629481132,
+        "narHash": "sha256-JHgasjPR0/J1J3DRm4KxM4zTyAj4IOJY8vIl75v/kPI=",
         "owner": "numtide",
         "repo": "flake-utils",
-        "rev": "5021eac20303a61fafe17224c087f5519baed54d",
+        "rev": "997f7efcb746a9c140ce1f13c72263189225f482",
         "type": "github"
       },
       "original": {
@@ -32,11 +32,11 @@
     },
     "nixpkgs": {
       "locked": {
-        "lastModified": 1608633860,
-        "narHash": "sha256-AGJfdJCR5jfIt8PqGiENXRqhthrS3Gxy8Wzb3Z2GsS4=",
+        "lastModified": 1629897889,
+        "narHash": "sha256-YoY/umk+NUtLFJgvTJkup6nLJb+sGEZ21hrupKTp7EI=",
         "owner": "NixOS",
         "repo": "nixpkgs",
-        "rev": "da1b28ab8f361fbe14dc539cd69ce1bfd015fd68",
+        "rev": "6248814b6892af7dc0cf973b49690fd102088e02",
         "type": "github"
       },
       "original": {

src/config.rs

@@ -12,6 +12,7 @@ pub struct Config {
 	pub from: PathBuf,
 	pub to: PathBuf,
 	pub matches: Vec<TranscodeMatch>,
+	pub jobs: Option<usize>,
 }
 
 #[derive(Debug)]
@@ -32,6 +33,12 @@ pub enum Transcode {
 		bitrate_type: BitrateType,
 	},
 
+	#[serde(rename = "flac")]
+	Flac {
+		#[serde(default = "default_flac_compression")]
+		compression: u8,
+	},
+
 	#[serde(rename = "mp3")]
 	Mp3 {
 		#[serde(default = "default_mp3_bitrate")]
@@ -40,13 +47,18 @@ pub enum Transcode {
 		#[serde(default = "bitrate_type_vbr")]
 		bitrate_type: BitrateType,
 	},
+
+	#[serde(rename = "copy")]
+	Copy,
 }
 
 impl Transcode {
 	pub fn extension(&self) -> &'static str {
 		match self {
 			Transcode::Opus { .. } => "opus",
+			Transcode::Flac { .. } => "flac",
 			Transcode::Mp3 { .. } => "mp3",
+			Transcode::Copy => "",
 		}
 	}
 }
@@ -55,6 +67,10 @@ fn default_opus_bitrate() -> u16 {
 	160
 }
 
+fn default_flac_compression() -> u8 {
+	5
+}
+
 fn bitrate_type_vbr() -> BitrateType {
 	BitrateType::Vbr
 }
@@ -112,7 +128,7 @@ pub fn config() -> Result<Config> {
 				.long("config")
 				.required(false)
 				.takes_value(true)
-                .help("path to an audio-conv config file, defaults to \"audio-conv.yaml\""),
+				.help("Path to an audio-conv config file, defaults to \"audio-conv.yaml\""),
 		)
 		.arg(
 			Arg::with_name("from")
@@ -120,7 +136,7 @@ pub fn config() -> Result<Config> {
 				.long("from")
 				.required(false)
 				.takes_value(true)
-                .help("from directory path"),
+				.help("\"from\" directory path"),
 		)
 		.arg(
 			Arg::with_name("to")
@@ -128,12 +144,20 @@ pub fn config() -> Result<Config> {
 				.long("to")
 				.required(false)
 				.takes_value(true)
-                .help("to directory path"),
+				.help("\"to\" directory path"),
+		)
+		.arg(
+			Arg::with_name("jobs")
+				.short("j")
+				.long("jobs")
+				.required(false)
+				.takes_value(true)
+				.help("Allow N jobs/transcodes at once. Defaults to number of logical cores"),
 		)
 		.subcommand(SubCommand::with_name("init").about("writes an example config"))
 		.get_matches();
 
-    let current_dir = std::env::current_dir().context("could not get current directory")?;
+	let current_dir = std::env::current_dir().context("Could not get current directory")?;
 
 	let config_path = arg_matches.value_of_os("config");
 	let force_load = config_path.is_some();
@@ -148,17 +172,17 @@ pub fn config() -> Result<Config> {
 			.create_new(true)
 			.open(&config_path)
 			.and_then(|mut f| f.write_all(std::include_bytes!("../example.audio-conv.yaml")))
-            .with_context(|| format!("unable to write config file to {}", config_path.display()))?;
+			.with_context(|| format!("Unable to write config file to {}", config_path.display()))?;
 
 		std::process::exit(0);
 	}
 
 	let config_dir = config_path
 		.parent()
-        .context("could not get parent directory of the config file")?;
+		.context("Could not get parent directory of the config file")?;
 
 	let config_file = load_config_file(&config_path)
-        .with_context(|| format!("failed loading config file \"{}\"", config_path.display()))?;
+		.with_context(|| format!("Failed loading config file {}", config_path.display()))?;
 
 	if force_load && config_file.is_none() {
 		return Err(Error::msg(format!(
@@ -170,7 +194,7 @@ pub fn config() -> Result<Config> {
 	let default_regex = RegexBuilder::new("\\.(flac|wav)$")
 		.case_insensitive(true)
 		.build()
-        .expect("failed compiling default match regex");
+		.expect("Failed compiling default match regex");
 
 	let transcode_matches = config_file
 		.as_ref()
@@ -183,8 +207,8 @@ pub fn config() -> Result<Config> {
 						let glob = GlobBuilder::new(glob)
 							.case_insensitive(true)
 							.build()
-                            .context("failed building glob")?;
-                        let regex = Regex::new(glob.regex()).context("failed compiling regex")?;
+							.context("Failed building glob")?;
+						let regex = Regex::new(glob.regex()).context("Failed compiling regex")?;
 						Ok(regex)
 					});
 
@@ -192,7 +216,7 @@ pub fn config() -> Result<Config> {
 						let regex = RegexBuilder::new(regex)
 							.case_insensitive(true)
 							.build()
-                            .context("failed compiling regex")?;
+							.context("Failed compiling regex")?;
 						Ok(regex)
 					});
 
@@ -204,7 +228,7 @@ pub fn config() -> Result<Config> {
 						let regex = RegexBuilder::new(&ext)
 							.case_insensitive(true)
 							.build()
-                            .context("failed compiling regex")?;
+							.context("Failed compiling regex")?;
 						Ok(regex)
 					});
 
@@ -247,7 +271,7 @@ pub fn config() -> Result<Config> {
 				})
 				.ok_or_else(|| Error::msg("\"from\" not configured"))?
 				.canonicalize()
-                .context("could not canonicalize \"from\" path")?
+				.context("Could not canonicalize \"from\" path")?
 		},
 		to: arg_matches
 			.value_of_os("to")
@@ -261,8 +285,26 @@ pub fn config() -> Result<Config> {
 			})
 			.ok_or_else(|| Error::msg("\"to\" not configured"))?
 			.canonicalize()
-            .context("could not canonicalize \"to\" path")?,
+			.context("Could not canonicalize \"to\" path")?,
 		matches: transcode_matches,
+		jobs: arg_matches
+			.value_of_os("jobs")
+			.map(|jobs_os_str| {
+				let jobs_str = jobs_os_str.to_str().with_context(|| {
+					// TODO: use `OsStr.display` when it lands
+					// https://github.com/rust-lang/rust/pull/80841
+					format!(
+						"Could not convert \"jobs\" argument to string due to invalid characters",
+					)
+				})?;
+				jobs_str.parse().with_context(|| {
+					format!(
+						"Could not parse \"jobs\" argument \"{}\" to a number",
+						&jobs_str
+					)
+				})
+			})
+			.transpose()?,
 	})
 }
 
@@ -273,6 +315,6 @@ fn load_config_file(path: &Path) -> Result<Option<ConfigFile>> {
 		Err(err) => return Err(Error::new(err)),
 	};
 	let config: ConfigFile =
-        serde_yaml::from_reader(&mut file).context("could not parse config file")?;
+		serde_yaml::from_reader(&mut file).context("Could not parse config file")?;
 	Ok(Some(config))
 }

src/main.rs

@@ -1,11 +1,11 @@
 mod config;
 mod ui;
 
-use crate::config::Config;
+use crate::config::{Config, Transcode};
 use anyhow::{Context, Error, Result};
-use futures::{future, pin_mut, prelude::*};
-use glib::{subclass::prelude::*, GBoxed, GString};
-use gstreamer::{gst_element_error, prelude::*, Element};
+use futures::{pin_mut, prelude::*};
+use glib::{GBoxed, GString};
+use gstreamer::{element_error, prelude::*, Element};
 use gstreamer_base::prelude::*;
 use std::{
 	borrow::Cow,
@@ -52,12 +52,12 @@ struct GErrorMessage {
 
 fn gmake<T: IsA<Element>>(factory_name: &str) -> Result<T> {
 	let res = gstreamer::ElementFactory::make(factory_name, None)
-        .with_context(|| format!("could not make gstreamer Element \"{}\"", factory_name))?
+		.with_context(|| format!("Could not make gstreamer Element \"{}\"", factory_name))?
 		.downcast()
 		.ok()
 		.with_context(|| {
 			format!(
-                "could not cast gstreamer Element \"{}\" into `{}`",
+				"Could not cast gstreamer Element \"{}\" into `{}`",
 				factory_name,
 				std::any::type_name::<T>()
 			)
@@ -68,7 +68,7 @@ fn gmake<T: IsA<Element>>(factory_name: &str) -> Result<T> {
 #[derive(Debug, Clone)]
 pub struct ConversionArgs {
 	rel_from_path: PathBuf,
-    transcode: config::Transcode,
+	transcode: Transcode,
 }
 
 fn get_conversion_args(config: &Config) -> impl Iterator<Item = Result<ConversionArgs>> + '_ {
@@ -97,7 +97,7 @@ fn get_conversion_args(config: &Config) -> impl Iterator<Item = Result<Conversio
 
 			let rel_path = e.path().strip_prefix(&config.from).with_context(|| {
 				format!(
-                    "unable to get relative path for {} from {}",
+					"Unable to get relative path for {} from {}",
 					e.path().display(),
 					config.from.display()
 				)
@@ -112,7 +112,10 @@ fn get_conversion_args(config: &Config) -> impl Iterator<Item = Result<Conversio
 					.map_err(Error::new)
 					.and_then(|md| md.modified().map_err(Error::new))
 					.with_context(|| {
-                        format!("unable to get mtime for from file {}", e.path().display())
+						format!(
+							"Unable to get mtime for \"from\" file {}",
+							e.path().display()
+						)
 					})?;
 				let to_mtime = to.metadata().and_then(|md| md.modified());
 				match to_mtime {
@@ -120,7 +123,7 @@ fn get_conversion_args(config: &Config) -> impl Iterator<Item = Result<Conversio
 					Err(err) if err.kind() == std::io::ErrorKind::NotFound => true,
 					Err(err) => {
 						return Err(err).with_context(|| {
-                            format!("unable to get mtime for to file {}", to.display())
+							format!("Unable to get mtime for \"to\" file {}", to.display())
 						})
 					}
 				}
@@ -147,15 +150,15 @@ async fn main() -> Result<()> {
 			let main_handle = async move {
 				let ok = task::spawn_local(main_loop(ui_queue))
 					.await
-                    .context("main task failed")??;
+					.context("Main task failed")??;
 				Result::<_>::Ok(ok)
 			};
 
 			let ui_handle = async move {
 				let ok = task::spawn_local(ui_fut)
 					.await
-                    .context("ui task failed")?
-                    .context("ui failed")?;
+					.context("Ui task failed")?
+					.context("Ui failed")?;
 				Result::<_>::Ok(ok)
 			};
 
@@ -168,20 +171,20 @@ async fn main() -> Result<()> {
 async fn main_loop(ui_queue: ui::MsgQueue) -> Result<()> {
 	let (config, conv_args) = task::spawn_blocking(|| -> Result<_> {
 		gstreamer::init()?;
-        let config = config::config().context("could not get the config")?;
+		let config = config::config().context("Could not get the config")?;
 
 		let conv_args = get_conversion_args(&config)
 			.collect::<Result<Vec<_>>>()
-            .context("failed loading dir structure")?;
+			.context("Failed loading dir structure")?;
 
 		Ok((config, conv_args))
 	})
 	.await
-    .context("init task failed")??;
+	.context("Init task failed")??;
 
 	let log_path = Path::new(".")
 		.canonicalize()
-        .context("unable to canonicalize path to log file")?
+		.context("Unable to canonicalize path to log file")?
 		.join("audio-conv.log");
 
 	ui_queue.push(ui::Msg::Init {
@@ -189,9 +192,11 @@ async fn main_loop(ui_queue: ui::MsgQueue) -> Result<()> {
 		log_path: log_path.clone(),
 	});
 
+	let concurrent_jobs = config.jobs.unwrap_or_else(|| num_cpus::get());
+
 	stream::iter(conv_args.into_iter().enumerate())
 		.map(Ok)
-        .try_for_each_concurrent(num_cpus::get(), |(i, args)| {
+		.try_for_each_concurrent(concurrent_jobs, |(i, args)| {
 			let config = &config;
 			let ui_queue = &ui_queue;
 			let log_path = &log_path;
@@ -206,7 +211,7 @@ async fn main_loop(ui_queue: ui::MsgQueue) -> Result<()> {
 					Ok(()) => ui_queue.push(ui::Msg::TaskEnd { id: i }),
 					Err(err) => {
 						let err = err.context(format!(
-                            "failed transcoding \"{}\"",
+							"Transcoding failed for {}",
 							args.rel_from_path.display()
 						));
 
@@ -264,15 +269,64 @@ async fn transcode(
 ) -> Result<()> {
 	let from_path = config.from.join(&args.rel_from_path);
 	let mut to_path = config.to.join(&args.rel_from_path);
-    to_path.set_extension(args.transcode.extension());
 
-    let file_src: Element = gmake("filesrc")?;
-    file_src.set_property("location", &path_to_gstring(&from_path))?;
+	fs::create_dir_all(
+		to_path
+			.parent()
+			.with_context(|| format!("Could not get parent dir for {}", to_path.display()))?,
+	)
+	.await?;
 
 	// encode into a tmp file first, then rename to actuall file name, that way we're writing
 	// "whole" files to the intended file path, ignoring partial files in the mtime check
 	let to_path_tmp = to_path.with_extension("tmp");
 
+	rm_file_on_err(&to_path_tmp, async {
+		match args.transcode {
+			Transcode::Copy => {
+				fs::copy(&from_path, &to_path_tmp).await.with_context(|| {
+					format!(
+						"Could not copy file from {} to {}",
+						from_path.display(),
+						to_path_tmp.display()
+					)
+				})?;
+			}
+			_ => {
+				to_path.set_extension(args.transcode.extension());
+
+				transcode_gstreamer(
+					&from_path,
+					&to_path_tmp,
+					args.transcode.clone(),
+					task_id,
+					queue,
+				)
+				.await?
+			}
+		}
+
+		fs::rename(&to_path_tmp, &to_path).await.with_context(|| {
+			format!(
+				"Could not rename temporary file {} to {}",
+				to_path_tmp.display(),
+				to_path.display()
+			)
+		})
+	})
+	.await
+}
+
+async fn transcode_gstreamer(
+	from_path: &Path,
+	to_path: &Path,
+	transcode: Transcode,
+	task_id: usize,
+	queue: &ui::MsgQueue,
+) -> Result<()> {
+	let file_src: Element = gmake("filesrc")?;
+	file_src.set_property("location", &path_to_gstring(&from_path))?;
+
 	let decodebin: Element = gmake("decodebin")?;
 
 	let src_elems: &[&Element] = &[&file_src, &decodebin];
@@ -285,10 +339,7 @@ async fn transcode(
 	// downgrade pipeline RC to a weak RC to break the reference cycle
 	let pipeline_weak = pipeline.downgrade();
 
-    let transcode_args = args.transcode.clone();
-
-    let to_path_tmp_clone = to_path_tmp.clone();
-
+	let to_path_clone = to_path.to_owned();
 	decodebin.connect_pad_added(move |decodebin, src_pad| {
 		let insert_sink = || -> Result<()> {
 			let pipeline = match pipeline_weak.upgrade() {
@@ -299,9 +350,9 @@ async fn transcode(
 				}
 			};
 
-            let is_audio = src_pad.get_current_caps().and_then(|caps| {
-                caps.get_structure(0).map(|s| {
-                    let name = s.get_name();
+			let is_audio = src_pad.current_caps().and_then(|caps| {
+				caps.structure(0).map(|s| {
+					let name = s.name();
 					name.starts_with("audio/")
 				})
 			});
@@ -309,7 +360,7 @@ async fn transcode(
 				None => {
 					return Err(Error::msg(format!(
 						"Failed to get media type from pad {}",
-                        src_pad.get_name()
+						src_pad.name()
 					)));
 				}
 				Some(false) => {
@@ -321,7 +372,7 @@ async fn transcode(
 
 			let resample: Element = gmake("audioresample")?;
 			// quality from 0 to 10
-            resample.set_property("quality", &7)?;
+			resample.set_property("quality", &10)?;
 
 			let mut dest_elems = vec![
 				resample,
@@ -329,8 +380,8 @@ async fn transcode(
 				gmake("audioconvert")?,
 			];
 
-            match &transcode_args {
-                config::Transcode::Opus {
+			match &transcode {
+				Transcode::Opus {
 					bitrate,
 					bitrate_type,
 				} => {
@@ -339,7 +390,7 @@ async fn transcode(
 						"bitrate",
 						&i32::from(*bitrate)
 							.checked_mul(1_000)
-                            .context("bitrate overflowed")?,
+							.context("Bitrate overflowed")?,
 					)?;
 					encoder.set_property_from_str(
 						"bitrate-type",
@@ -352,7 +403,14 @@ async fn transcode(
 					dest_elems.push(encoder);
 					dest_elems.push(gmake("oggmux")?);
 				}
-                config::Transcode::Mp3 {
+
+				Transcode::Flac { compression } => {
+					let encoder: Element = gmake("flacenc")?;
+					encoder.set_property_from_str("quality", &compression.to_string());
+					dest_elems.push(encoder);
+				}
+
+				Transcode::Mp3 {
 					bitrate,
 					bitrate_type,
 				} => {
@@ -371,10 +429,15 @@ async fn transcode(
 					dest_elems.push(encoder);
 					dest_elems.push(gmake("id3v2mux")?);
 				}
+
+				Transcode::Copy => {
+					// already handled outside this fn
+					unreachable!();
+				}
 			};
 
 			let file_dest: gstreamer_base::BaseSink = gmake("filesink")?;
-            file_dest.set_property("location", &path_to_gstring(&to_path_tmp_clone))?;
+			file_dest.set_property("location", &path_to_gstring(&to_path_clone))?;
 			file_dest.set_sync(false);
 			dest_elems.push(file_dest.upcast());
 
@@ -389,7 +452,7 @@ async fn transcode(
 			let sink_pad = dest_elems
 				.get(0)
 				.unwrap()
-                .get_static_pad("sink")
+				.static_pad("sink")
 				.expect("1. dest element has no sinkpad");
 			src_pad.link(&sink_pad)?;
 
@@ -401,7 +464,7 @@ async fn transcode(
 				.field("error", &GBoxErrorWrapper::new(err))
 				.build();
 
-            gst_element_error!(
+			element_error!(
 				decodebin,
 				gstreamer::LibraryError::Failed,
 				("Failed to insert sink"),
@@ -410,16 +473,8 @@ async fn transcode(
 		}
 	});
 
-    let bus = pipeline.get_bus().context("pipe get bus")?;
+	let bus = pipeline.bus().context("Could not get bus for pipeline")?;
 
-    fs::create_dir_all(
-        to_path
-            .parent()
-            .with_context(|| format!("could not get parent dir for {}", to_path.display()))?,
-    )
-    .await?;
-
-    rm_file_on_err(&to_path_tmp, async {
 	pipeline
 		.set_state(gstreamer::State::Playing)
 		.context("Unable to set the pipeline to the `Playing` state")?;
@@ -439,37 +494,43 @@ async fn transcode(
 						Ok(false)
 					}
 					MessageView::Error(err) => {
-                            pipeline.set_state(gstreamer::State::Null).context(
-                                "Unable to set the pipeline to the `Null` state, after error",
-                            )?;
+						let pipe_stop_res = pipeline.set_state(gstreamer::State::Null);
 
-                            let err = err
-                                .get_details()
+						let err: Error = err
+							.details()
 							.and_then(|details| {
-                                    if details.get_name() != "error-details" {
+								if details.name() != "error-details" {
 									return None;
 								}
 
 								let err = details
 									.get::<&GBoxErrorWrapper>("error")
 									.unwrap()
-                                        .map(|err| err.clone().into())
-                                        .expect("error-details message without actual error");
+									.clone()
+									.into();
 								Some(err)
 							})
 							.unwrap_or_else(|| {
 								GErrorMessage {
 									src: msg
-                                            .get_src()
-                                            .map(|s| String::from(s.get_path_string()))
+										.src()
+										.map(|s| String::from(s.path_string()))
 										.unwrap_or_else(|| String::from("None")),
-                                        error: err.get_error().to_string(),
-                                        debug: err.get_debug(),
-                                        source: err.get_error(),
+									error: err.error().to_string(),
+									debug: err.debug(),
+									source: err.error(),
 								}
 								.into()
 							});
+
+						if let Err(pipe_err) = pipe_stop_res {
+							let err = err.context(pipe_err).context(
+								"Unable to set the pipeline to the `Null` state, after error",
+							);
 							Err(err)
+						} else {
+							Err(err)
+						}
 					}
 					_ => Ok(true),
 				}
@@ -482,8 +543,7 @@ async fn transcode(
 				}
 			})
 			.try_for_each(|_| futures::future::ready(Ok(())))
-                .await
-                .context("failed converting")?;
+			.await?;
 
 		Result::<_>::Ok(())
 	};
@@ -498,7 +558,7 @@ async fn transcode(
 
 			let dur = decodebin
 				.query_duration::<ClockTime>()
-                    .and_then(|time| time.nanoseconds());
+				.map(|time| time.nseconds());
 
 			let ratio = dur.and_then(|dur| {
 				if dur == 0 {
@@ -507,11 +567,11 @@ async fn transcode(
 
 				let pos = decodebin
 					.query_position::<ClockTime>()
-                        .and_then(|time| time.nanoseconds());
+					.map(|time| time.nseconds());
 
 				pos.map(|pos| {
 					let ratio = pos as f64 / dur as f64;
-                        ratio.max(0.0).min(1.0)
+					ratio.clamp(0.0, 1.0)
 				})
 			});
 
@@ -533,25 +593,21 @@ async fn transcode(
 		.set_state(gstreamer::State::Null)
 		.context("Unable to set the pipeline to the `Null` state")?;
 
-        fs::rename(&to_path_tmp, &to_path).await?;
-
 	Ok(())
-    })
-    .await
 }
 
-async fn rm_file_on_err<F, T>(path: &Path, f: F) -> F::Output
+async fn rm_file_on_err<F, T>(path: &Path, f: F) -> Result<T>
 where
 	F: Future<Output = Result<T>>,
 {
 	match f.await {
 		Err(err) => match fs::remove_file(path).await {
-            Ok(..) => Err(err),
+			Ok(()) => Err(err),
 			Err(fs_err) if fs_err.kind() == std::io::ErrorKind::NotFound => Err(err),
 			Err(fs_err) => {
 				let err = err
 					.context(fs_err)
-                    .context(format!("removing {} failed", path.display()));
+					.context(format!("Removing file {} failed", path.display()));
 				Err(err)
 			}
 		},

src/ui.rs

@@ -88,7 +88,7 @@ impl State {
 			Msg::TaskEnd { id } => {
 				self.running_tasks
 					.remove(&id)
-                    .context("unable to remove finished task; could't find task")?;
+					.context("Unable to remove finished task; could't find task")?;
 				self.ended_tasks += 1;
 			}
 			Msg::TaskProgress { id, ratio } => {
@@ -102,7 +102,7 @@ impl State {
 				// TODO
 				self.running_tasks
 					.remove(&id)
-                    .context("unable to remove errored task; could't find task")?;
+					.context("Unable to remove errored task; could't find task")?;
 				self.ended_tasks += 1;
 				self.has_errored = true;
 			}
@@ -136,7 +136,7 @@ impl State {
 		running_tasks.sort_by_key(|task| task.id);
 
 		if !self.has_rendered {
-            self.terminal.clear().context("cleaning ui failed")?;
+			self.terminal.clear().context("Clearing ui failed")?;
 			self.has_rendered = true;
 		}
 
@@ -222,7 +222,7 @@ impl State {
 					chunks[1],
 				);
 			})
-            .context("rendering ui failed")?;
+			.context("Rendering ui failed")?;
 
 		Ok(())
 	}
@@ -267,8 +267,8 @@ pub fn init() -> (MsgQueue, impl Future<Output = Result<()>>) {
 				}
 			})
 			.await
-            .context("ui update task failed")?
-            .context("ui update failed")?;
+			.context("Ui update task failed")?
+			.context("Ui update failed")?;
 
 			match render_res {
 				Some(s) => wrapped = Some(s),