moved to claxon

2 files changed, 97 insertions, 219 deletions

diff --git a/src/files.rs b/src/files.rs
index 8053da2..f9352d3 100644
--- a/src/files.rs
+++ b/src/files.rs
@@ -13,7 +13,7 @@ use tokio::task::JoinSet;
 use tokio_util::sync::CancellationToken;
 use walkdir::WalkDir;
 
-use crate::{db::Database, flac::encode_file};
+use crate::{db::Database, flac::handle_encode};
 
 #[cfg(not(test))]
 const BAR_TEMPLATE: &str = "{msg:<} [{wide_bar:.green/cyan}] Elapsed: {elapsed} {pos:>7}/{len:7}";
@@ -169,7 +169,7 @@ pub async fn reencode_files(conn: &Database, canceltoken: CancellationToken) ->
                         Ok(())
                     }
                     res = async {
-                        if let Err(error) = tokio::task::spawn_blocking(move || encode_file(file)).await? {
+                        if let Err(error) = tokio::task::spawn_blocking(move || handle_encode(file)).await? {
                             let _ = std::fs::remove_file(filename.with_extension("tmp.metadata_edit"));
                             let _ = std::fs::remove_file(filename.with_extension("tmp"));
                             return Err(anyhow!(FileError::new(&filename, error)));
diff --git a/src/flac.rs b/src/flac.rs
index 05d915f..823f4e4 100644
--- a/src/flac.rs
+++ b/src/flac.rs
@@ -1,208 +1,114 @@
 use anyhow::{Result, anyhow};
-use flac_bound::{FlacEncoder, WriteWrapper};
+use claxon::{FlacReader, FlacReaderOptions};
+use flac_bound::FlacEncoder;
 use md5::{Digest, Md5};
 use metaflac::{Block, Tag};
-use std::{
-    fs::File,
-    path::{Path, PathBuf},
-};
-use symphonia::core::{
-    audio::{Audio, GenericAudioBufferRef},
-    codecs::audio::AudioDecoder,
-    formats::{FormatOptions, FormatReader, TrackType, probe::Hint},
-    io::MediaSourceStream,
-    meta::MetadataOptions,
-};
+use std::path::Path;
 
 pub const CURRENT_VENDOR: &str = "reference libFLAC 1.5.0 20250211";
 
-type BoxedFormatReader = Box<dyn FormatReader>;
-type BoxedAudioDecoder = Box<dyn AudioDecoder + 'static>;
+fn write_tags(filename: impl AsRef<Path>, hash: Vec<u8>) -> Result<()> {
+    let tags = Tag::read_from_path(&filename)?;
+    let temp_name = filename.as_ref().with_extension("tmp");
+    let mut output = Tag::read_from_path(&temp_name)?;
 
-struct StreamConfig {
-    channels: u32,
-    bits_per_sample: Bps,
-    sample_rate: u32,
-}
-
-enum Bps {
-    _16,
-    _24,
-    _32,
-}
+    let mut streaminfo = tags.get_streaminfo().unwrap().clone();
 
-impl Bps {
-    fn new(num: u32) -> Result<Self> {
-        match num {
-            16 => Ok(Bps::_16),
-            24 => Ok(Bps::_24),
-            32 => Ok(Bps::_32),
-            _ => Err(anyhow!("Invalid BPS")),
-        }
-    }
+    streaminfo.md5 = hash;
+    output.set_streaminfo(streaminfo);
 
-    fn value(&self) -> u32 {
-        match self {
-            Bps::_16 => 16,
-            Bps::_24 => 24,
-            Bps::_32 => 32,
+    for block in tags.blocks() {
+        match block {
+            Block::VorbisComment(comment) => {
+                for (key, val) in comment.comments.clone() {
+                    if key != "ENCODER" {
+                        output.set_vorbis(key, val);
+                    }
+                }
+            }
+            Block::StreamInfo(_) | Block::Padding(_) => {}
+            _ => output.push_block(block.clone()),
         }
     }
-}
 
-struct FileEncoder {
-    filename: PathBuf,
-    streamdata: StreamConfig,
-    format: BoxedFormatReader,
-    decoder: BoxedAudioDecoder,
+    output.write_to_path(temp_name)?;
+    Ok(())
 }
 
-impl FileEncoder {
-    fn new(file: impl AsRef<Path>) -> Result<Self> {
-        let (format, decoder, config) = init_decoder(&file)?;
-        Ok(FileEncoder {
-            filename: file.as_ref().to_path_buf(),
-            streamdata: config,
-            format,
-            decoder,
-        })
-    }
-
-    fn temp_name(&self) -> PathBuf {
-        self.filename.clone().with_extension("tmp")
+fn encode_file(filename: impl AsRef<Path>) -> Result<()> {
+    let temp_name = filename.as_ref().with_extension("tmp");
+    if temp_name.exists() {
+        std::fs::remove_file(&temp_name)?;
     }
+    let mut decoder = FlacReader::open(&filename)?;
+    let streaminfo = decoder.streaminfo();
 
-    fn encode(&mut self, mut encoder: FlacEncoder) -> Result<Vec<u8>> {
-        let mut buffer: Vec<i32> = Vec::new();
-        let mut hasher = Md5::new();
-        let track_id = self.format.default_track(TrackType::Audio).unwrap().id;
-        let offset = self.streamdata.bits_per_sample.value();
-
-        loop {
-            let packet = match self.format.next_packet() {
-                Ok(Some(packet)) => packet,
-                Ok(None) => break,
-                Err(error) => return Err(error.into()),
-            };
+    let mut encoder = FlacEncoder::new()
+        .unwrap()
+        .channels(streaminfo.channels)
+        .bits_per_sample(streaminfo.bits_per_sample)
+        .sample_rate(streaminfo.sample_rate)
+        .compression_level(8)
+        .verify(false)
+        .init_file(&temp_name)
+        .unwrap();
 
-            if packet.track_id() != track_id {
-                continue;
-            }
+    let mut hasher = Md5::new();
 
-            if let GenericAudioBufferRef::S32(buf) = self.decoder.decode(&packet)? {
-                for sample in buf.iter_interleaved() {
-                    let real_sample = sample >> (32 - offset);
-                    hasher.update(real_sample.to_le_bytes());
-                    buffer.push(real_sample);
+    let samples = decoder
+        .samples()
+        .map(|res| {
+            let sample = res.unwrap();
+            match streaminfo.bits_per_sample {
+                16 => {
+                    hasher.update(i16::try_from(sample).unwrap().to_le_bytes());
                 }
-                encoder
-                    .process_interleaved(&buffer, buf.samples_planar() as u32)
-                    .unwrap();
-                buffer.clear();
-            } else {
-                return Err(anyhow!("unsupported codec"));
-            }
-        }
-
-        if let Err(enc) = encoder.finish() {
-            return Err(anyhow!("Encoding failed:\t{:?}", enc.state()));
-        }
-
-        Ok(hasher.finalize().to_vec())
-    }
-
-    fn write_tags(&self, hash: Vec<u8>) -> Result<()> {
-        let tags = Tag::read_from_path(&self.filename)?;
-        let mut output = Tag::read_from_path(self.temp_name())?;
-
-        let mut streaminfo = tags.get_streaminfo().unwrap().clone();
-
-        streaminfo.md5 = hash;
-        output.set_streaminfo(streaminfo);
-
-        for block in tags.blocks() {
-            match block {
-                Block::VorbisComment(comment) => {
-                    for (key, val) in comment.comments.clone() {
-                        if key != "ENCODER" {
-                            output.set_vorbis(key, val);
-                        }
-                    }
+                24 => {
+                    hasher.update(i24::i24::try_from_i32(sample).unwrap().to_le_bytes());
+                }
+                32 => {
+                    hasher.update(sample.to_le_bytes());
                 }
-                Block::StreamInfo(_) | Block::Padding(_) => {}
-                _ => output.push_block(block.clone()),
+                _ => {}
             }
-        }
-
-        output.write_to_path(self.temp_name())?;
-        Ok(())
-    }
-}
-
-fn init_decoder(
-    filename: impl AsRef<Path>,
-) -> Result<(BoxedFormatReader, BoxedAudioDecoder, StreamConfig)> {
-    let src = std::fs::File::open(filename)?;
-    let mss = MediaSourceStream::new(Box::new(src), Default::default());
-    let mut hint = Hint::new();
-    hint.with_extension("flac");
-
-    let format_opts: FormatOptions = Default::default();
-    let metadata_opts: MetadataOptions = Default::default();
-
-    let format = symphonia::default::get_probe()
-        .probe(&hint, mss, format_opts, metadata_opts)
-        .unwrap();
-
-    let track = format.default_track(TrackType::Audio).unwrap();
+            sample
+        })
+        .collect::<Vec<i32>>();
 
-    let decoder = symphonia::default::get_codecs()
-        .make_audio_decoder(
-            track.codec_params.as_ref().unwrap().audio().unwrap(),
-            &Default::default(),
-        )
+    encoder
+        .process_interleaved(&samples, samples.len() as u32 / streaminfo.channels)
         .unwrap();
 
-    let params = track.codec_params.as_ref().unwrap().audio().unwrap();
-
-    let config = StreamConfig {
-        channels: u32::try_from(params.channels.as_ref().unwrap().count()).unwrap(),
-        bits_per_sample: Bps::new(params.bits_per_sample.unwrap())?,
-        sample_rate: params.sample_rate.unwrap(),
-    };
+    if let Err(enc) = encoder.finish() {
+        return Err(anyhow!("Encoding failed:\t{:?}", enc.state()));
+    }
 
-    Ok((format, decoder, config))
+    let hash = hasher.finalize().to_vec();
+    write_tags(&filename, hash)?;
+    std::fs::rename(temp_name, filename)?;
+    Ok(())
 }
 
-pub fn encode_file(filename: impl AsRef<Path>) -> Result<()> {
-    let mut filencoder = FileEncoder::new(filename)?;
-    let temp_name = filencoder.temp_name();
-
-    if temp_name.exists() {
-        std::fs::remove_file(&temp_name)?;
+pub fn handle_encode(filename: impl AsRef<Path>) -> Result<()> {
+    if let Err(error) = encode_file(&filename) {
+        let _ = std::fs::remove_file(filename.as_ref().with_extension("tmp"));
+        Err(error)
+    } else {
+        Ok(())
     }
-
-    let mut outf = File::create(temp_name)?;
-    let mut outw = WriteWrapper(&mut outf);
-    let enc = FlacEncoder::new()
-        .unwrap()
-        .channels(filencoder.streamdata.channels)
-        .bits_per_sample(filencoder.streamdata.bits_per_sample.value())
-        .sample_rate(filencoder.streamdata.sample_rate)
-        .compression_level(8)
-        .verify(false)
-        .init_write(&mut outw)
-        .unwrap();
-
-    let hash = filencoder.encode(enc)?;
-    filencoder.write_tags(hash)?;
-    std::fs::rename(filencoder.temp_name(), filencoder.filename)?;
-    Ok(())
 }
 
 pub fn get_vendor(file: impl AsRef<Path>) -> Result<String> {
-    if let Some(vorbis) = Tag::read_from_path(file)?.vorbis_comments() {
-        Ok(vorbis.vendor_string.to_owned())
+    if let Some(vendor) = FlacReader::open_ext(
+        file,
+        FlacReaderOptions {
+            metadata_only: true,
+            read_vorbis_comment: true,
+        },
+    )?
+    .vendor()
+    {
+        Ok(vendor.to_string())
     } else {
         Err(anyhow!("Vendor string not found"))
     }
@@ -211,7 +117,6 @@ pub fn get_vendor(file: impl AsRef<Path>) -> Result<String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use metaflac::Tag;
 
     #[test]
     fn bit16() {
@@ -219,20 +124,11 @@ mod tests {
         let tempname = "16bit.flac.temp";
         std::fs::copy(name, tempname).unwrap();
         encode_file(name).unwrap();
-        let target_md5 = Tag::read_from_path(tempname)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        let encoded_md5 = Tag::read_from_path(name)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        std::fs::remove_file(tempname).unwrap();
-        assert_eq!(target_md5, encoded_md5);
+        let target_md5 = FlacReader::open(tempname).unwrap().streaminfo().md5sum;
+        let temp_md5 = FlacReader::open(name).unwrap().streaminfo().md5sum;
+
+        std::fs::rename(tempname, name).unwrap();
+        assert_eq!(target_md5, temp_md5);
     }
 
     #[test]
@@ -241,20 +137,11 @@ mod tests {
         let tempname = "24bit.flac.temp";
         std::fs::copy(name, tempname).unwrap();
         encode_file(name).unwrap();
-        let target_md5 = Tag::read_from_path(tempname)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        let encoded_md5 = Tag::read_from_path(name)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        std::fs::remove_file(tempname).unwrap();
-        assert_eq!(target_md5, encoded_md5);
+        let target_md5 = FlacReader::open(tempname).unwrap().streaminfo().md5sum;
+        let temp_md5 = FlacReader::open(name).unwrap().streaminfo().md5sum;
+
+        std::fs::rename(tempname, name).unwrap();
+        assert_eq!(target_md5, temp_md5);
     }
 
     #[test]
@@ -263,19 +150,10 @@ mod tests {
         let tempname = "32bit.flac.temp";
         std::fs::copy(name, tempname).unwrap();
         encode_file(name).unwrap();
-        let target_md5 = Tag::read_from_path(tempname)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        let encoded_md5 = Tag::read_from_path(name)
-            .unwrap()
-            .get_streaminfo()
-            .unwrap()
-            .md5
-            .clone();
-        std::fs::remove_file(tempname).unwrap();
-        assert_eq!(target_md5, encoded_md5);
+        let target_md5 = FlacReader::open(tempname).unwrap().streaminfo().md5sum;
+        let temp_md5 = FlacReader::open(name).unwrap().streaminfo().md5sum;
+
+        std::fs::rename(tempname, name).unwrap();
+        assert_eq!(target_md5, temp_md5);
     }
 }