diff --git a/CHANGELOG.md b/CHANGELOG.md index e0f149dec..94b004b8a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -11,6 +11,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Added +- Added `FixedSource` which mirrors `Source` but does not allow the sample rate or + channel count to change. +- Added `ConstSource` which is like `FixedSource` except the sample rate and + channel count are fixed at compile time. - Added `Skippable::skipped` function to check if the inner source was skipped. - All sources now implement `ExactSizeIterator` when their inner source does. - All sources now implement `Iterator::size_hint()`. @@ -21,6 +25,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Changed +- Breaking: `Microphone` now implements `FixedSource` - Breaking: `Done` now calls a callback instead of decrementing an `Arc`. - Updated `cpal` to v0.18. - Clarified `Source::current_span_len()` documentation to specify it returns total span length. diff --git a/Cargo.toml b/Cargo.toml index 1dfac553e..88ce817d3 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -250,7 +250,7 @@ required-features = ["playback", "wav"] [[example]] name = "microphone" -required-features = ["playback", "recording", "wav_output"] +required-features = ["playback", "recording"] [[example]] name = "mix_multiple_sources" diff --git a/README.md b/README.md index ccc96feaf..8586483ff 100644 --- a/README.md +++ b/README.md @@ -4,6 +4,14 @@ [![Crates.io Downloads](https://img.shields.io/crates/d/rodio.svg)](https://crates.io/crates/rodio) [![Build Status](https://github.com/RustAudio/rodio/workflows/CI/badge.svg)](https://github.com/RustAudio/rodio/actions) +> [!WARNING] +> We are currently rewriting Rodio's core audio engine. The current state should +> be working but under documented. This is expected to take at least a month. +> For the latest unreleased work pre engine rewrite see [this +> checkout](https://github.com/RustAudio/rodio/tree/c5f1e94290922fe4ee963ce6f85754ea6ba36243). +> To follow our progress see the [tracking +> issue](https://github.com/RustAudio/rodio/issues/901) + Rust playback library. Playback is handled by [cpal](https://github.com/RustAudio/cpal). Format decoding is handled by [Symphonia](https://github.com/pdeljanov/Symphonia) by default, or by optional format-specific decoders: diff --git a/examples/microphone.rs b/examples/microphone.rs index e196208a0..9568116a1 100644 --- a/examples/microphone.rs +++ b/examples/microphone.rs @@ -1,6 +1,6 @@ use inquire::Select; use rodio::microphone::{self, MicrophoneBuilder}; -use rodio::Source; +use rodio::FixedSource; use std::error::Error; use std::thread; use std::time::Duration; @@ -22,7 +22,7 @@ fn main() -> Result<(), Box> { println!("Playing the recording"); let mut output = rodio::DeviceSinkBuilder::open_default_sink()?; - output.mixer().add(recording); + output.mixer().add(recording.into_dynamic_source()); thread::sleep(Duration::from_secs(5)); diff --git a/src/buffer.rs b/src/buffer.rs index 187a6b79b..f9c03a412 100644 --- a/src/buffer.rs +++ b/src/buffer.rs @@ -12,9 +12,8 @@ //! use crate::common::{ChannelCount, SampleRate}; -use crate::math::{duration_to_float, NANOS_PER_SEC}; use crate::source::{SeekError, UniformSourceIterator}; -use crate::{Float, Sample, Source}; +use crate::{Sample, Source}; use std::sync::Arc; use std::time::Duration; @@ -25,37 +24,19 @@ pub struct SamplesBuffer { pos: usize, channels: ChannelCount, sample_rate: SampleRate, - duration: Duration, } impl SamplesBuffer { /// Builds a new `SamplesBuffer`. - /// - /// # Panics - /// - /// - Panics if the samples rate is zero. - /// - Panics if the length of the buffer is larger than approximately 16 billion elements. - /// This is because the calculation of the duration would overflow. - /// pub fn new(channels: ChannelCount, sample_rate: SampleRate, data: D) -> Self where D: Into>, { - let data: Arc<[Sample]> = data.into().into(); - let duration_ns = NANOS_PER_SEC.checked_mul(data.len() as u64).unwrap() - / sample_rate.get() as u64 - / channels.get() as u64; - let duration = Duration::new( - duration_ns / NANOS_PER_SEC, - (duration_ns % NANOS_PER_SEC) as u32, - ); - Self { - data, + data: data.into().into(), pos: 0, channels, sample_rate, - duration, } } @@ -91,50 +72,11 @@ impl Source for SamplesBuffer { self.sample_rate } - #[inline] - fn total_duration(&self) -> Option { - Some(self.duration) - } - - /// This jumps in memory till the sample for `pos`. - #[inline] - fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { - // This is fast because all the samples are in memory already - // and due to the constant sample_rate we can jump to the right - // sample directly. - - let curr_channel = self.pos % self.channels().get() as usize; - let new_pos = duration_to_float(pos) - * self.sample_rate().get() as Float - * self.channels().get() as Float; - // saturate pos at the end of the source - let new_pos = new_pos as usize; - let new_pos = new_pos.min(self.data.len()); - - // make sure the next sample is for the right channel - let new_pos = new_pos.next_multiple_of(self.channels().get() as usize); - let new_pos = new_pos - curr_channel; - - self.pos = new_pos; - Ok(()) - } + crate::common::source::buffer::source_impl! {} } impl Iterator for SamplesBuffer { - type Item = Sample; - - #[inline] - fn next(&mut self) -> Option { - let sample = self.data.get(self.pos)?; - self.pos += 1; - Some(*sample) - } - - #[inline] - fn size_hint(&self) -> (usize, Option) { - let remaining = self.data.len() - self.pos; - (remaining, Some(remaining)) - } + crate::common::source::buffer::iter_impl! {} } impl ExactSizeIterator for SamplesBuffer {} diff --git a/src/common.rs b/src/common.rs index d1ed01366..edc44b7f8 100644 --- a/src/common.rs +++ b/src/common.rs @@ -3,6 +3,8 @@ use std::num::NonZero; use crate::math::nz; +pub(crate) mod source; + /// Sample rate (a frame rate or samples per second per channel). pub type SampleRate = NonZero; diff --git a/src/common/source.rs b/src/common/source.rs new file mode 100644 index 000000000..ec6b65347 --- /dev/null +++ b/src/common/source.rs @@ -0,0 +1,44 @@ +//! Code shared between source types. +//! +//! Since we have three source types there is a lot of code duplication. We +//! combat that by placing whatever can be shared here. +//! +//! This can be: +//! - shared types like error enums +//! - shared free functions +//! - shared members / impl blocks through macro_rules +//! +//! # Note +//! Effects are defined through a macro and do not need this kind of +//! deduplication +//! +//! This modules structure mirrors that of what it deduplicates. For example +//! the code shared between [fixed_source::chain] and [const_source::chain] is in +//! common/source/chain.rs + +pub(crate) mod buffer; +pub(crate) mod chain; + +macro_rules! add_inner_accessors { + ($inner:ident) => { + /// Get immutable access to the input to this source + #[inline] + pub fn inner(&self) -> &S { + &self.$inner + } + + /// Returns a mutable reference to the inner source. + #[inline] + pub fn inner_mut(&mut self) -> &mut S { + &mut self.$inner + } + + /// Returns the inner source. + #[inline] + pub fn into_inner(self) -> S { + self.$inner + } + }; +} + +pub(crate) use add_inner_accessors; diff --git a/src/common/source/buffer.rs b/src/common/source/buffer.rs new file mode 100644 index 000000000..49a625b81 --- /dev/null +++ b/src/common/source/buffer.rs @@ -0,0 +1,66 @@ +macro_rules! source_impl { + () => { + /// # Panics + /// If the length of the buffer is larger than approximately 16 billion elements. + /// This is because the calculation of the duration would overflow. + #[inline] + fn total_duration(&self) -> Option { + use crate::math::NANOS_PER_SEC; + + let duration_ns = NANOS_PER_SEC + .checked_mul(self.data.len() as u64) + .expect("slices longer then 16 billion elements are not supported") + / self.sample_rate().get() as u64 + / self.channels().get() as u64; + let duration = Duration::new( + duration_ns / NANOS_PER_SEC, + (duration_ns % NANOS_PER_SEC) as u32, + ); + + Some(duration) + } + + /// This jumps in memory to the sample corresponding to `pos`. + #[inline] + fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { + // This is fast because all the samples are in memory already + // and due to the constant sample_rate we can jump to the right + // sample directly. + + let curr_channel = self.pos % self.channels().get() as usize; + let new_pos = crate::math::duration_to_float(pos) + * self.sample_rate().get() as crate::Float + * self.channels().get() as crate::Float; + // saturate pos at the end of the source + let new_pos = new_pos as usize; + let new_pos = new_pos.min(self.data.len()); + + // make sure the next sample is for the right channel + let new_pos = new_pos.next_multiple_of(self.channels().get() as usize); + let new_pos = new_pos - curr_channel; + + self.pos = new_pos; + Ok(()) + } + }; +} + +macro_rules! iter_impl { + () => { + type Item = Sample; + #[inline] + fn next(&mut self) -> Option { + let sample = self.data.get(self.pos)?; + self.pos += 1; + Some(*sample) + } + #[inline] + fn size_hint(&self) -> (usize, Option) { + let remaining = self.data.len() - self.pos; + (remaining, Some(remaining)) + } + }; +} + +pub(crate) use iter_impl; +pub(crate) use source_impl; diff --git a/src/common/source/chain.rs b/src/common/source/chain.rs new file mode 100644 index 000000000..65f370b3d --- /dev/null +++ b/src/common/source/chain.rs @@ -0,0 +1,94 @@ +use crate::source::SeekError; + +#[derive(Debug, thiserror::Error)] +pub enum ChainSeekError { + #[error("Could not get duration of first source ({ty}")] + NoTotalDurationForFirst { ty: &'static str }, + #[error("Could not seek in first source")] + FailedToSeekInFirst { + ty: &'static str, + #[source] + error: SeekError, + }, + #[error("Could not seek in second source")] + FailedToSeekInSecond { + ty: &'static str, + #[source] + error: SeekError, + }, +} + +macro_rules! source_impl { + () => { + fn channels(&self) -> crate::ChannelCount { + self.first.channels() + } + + fn sample_rate(&self) -> crate::SampleRate { + self.first.sample_rate() + } + + fn total_duration(&self) -> Option { + self.first + .total_duration() + .and_then(|d| self.second.total_duration().map(|d2| d2 + d)) + } + + fn try_seek(&mut self, pos: std::time::Duration) -> Result<(), crate::source::SeekError> { + use crate::source::SeekError; + use std::any::type_name_of_val; + use std::sync::Arc; + + let Some(first) = self.first.total_duration() else { + return Err(ChainSeekError::NoTotalDurationForFirst { + ty: type_name_of_val(&self.first), + }) + .map_err(Arc::new) + .map_err(|e| SeekError::Other(e)); + }; + + if pos < first { + self.first + .try_seek(pos) + .map_err(|error| ChainSeekError::FailedToSeekInFirst { + ty: type_name_of_val(&self.first), + error, + }) + .map_err(Arc::new) + .map_err(|e| SeekError::Other(e)) + } else { + self.second + .try_seek(pos) + .map_err(|error| ChainSeekError::FailedToSeekInSecond { + ty: type_name_of_val(&self.second), + error, + }) + .map_err(Arc::new) + .map_err(|e| SeekError::Other(e)) + } + } + }; +} + +macro_rules! iter_impl { + () => { + type Item = Sample; + + fn next(&mut self) -> Option { + if self.playing_inner { + match self.first.next() { + Some(sample) => Some(sample), + None => { + self.playing_inner = false; + self.second.next() + } + } + } else { + self.second.next() + } + } + }; +} + +pub(crate) use iter_impl; +pub(crate) use source_impl; diff --git a/src/const_source.rs b/src/const_source.rs new file mode 100644 index 000000000..29120d1d2 --- /dev/null +++ b/src/const_source.rs @@ -0,0 +1,224 @@ +//! A sound source that has it's sample rate and number of channels fixed at compile time. +//! Some practical examples: +//! - An effect performed by a neural network trained on a specific channel +//! count and sample rate +//! - A custom audio source streaming in at a pre-determined fixed sample rate. +//! We use this in our VoIP example. +//! +//! A const pipeline _may_ also be optimized more by the compiler as many +//! branches are known at compile time, like when converting from one channel +//! count to another. +use std::num::NonZeroU16; +use std::num::NonZeroU32; +use std::time::Duration; + +use crate::source::SeekError; +use crate::ChannelCount; +use crate::FixedSource; +use crate::Sample; +use crate::SampleRate; +use crate::Source as DynamicSource; // Source will (probably) be renamed to this later + +mod buffer; +mod chain; +mod conversions; + +pub use buffer::SamplesBuffer; +pub use chain::SourceChain; +pub use conversions::channel_count::ChannelConvertor; + +/// A source which sample rate and channel count are fixed at compile time. +pub trait ConstSource: Iterator { + /// Returns the sample rate which is the first generic (SR) of a ConstSource + fn sample_rate(&self) -> SampleRate { + const { NonZeroU32::new(SR).expect("SampleRate must be > 0") } + } + /// Returns the channel count which is the second generic (CH) of a ConstSource + fn channels(&self) -> ChannelCount { + const { NonZeroU16::new(CH).expect("Channel count must be > 0") } + } + + #[doc = include_str!("docs/total_duration.md")] + fn total_duration(&self) -> Option; + + #[allow(unused_variables)] + #[doc = include_str!("docs/try_seek.md")] + fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { + Err(SeekError::NotSupported { + underlying_source: std::any::type_name::(), + }) + } + + /// Convert from the current channel count to `CH_OUT`. + fn with_channel_count(self) -> ChannelConvertor + where + Self: Sized, + { + ChannelConvertor::new(self) + } + + /// Use this const source as if it's a dynamic source. You generally do not + /// want to do this since there are less effects for dynamic sources and + /// those that are available can not be implemented as efficient. This is + /// therefore purely provided for backwards compatibility. + fn into_dynamic_source(self) -> IntoDynamicSource + where + Self: Sized, + { + IntoDynamicSource { inner: self } + } + + /// Use this const source as if it's a fixed source which is generally + /// easier to work with since it drops the generics. The same effects are + /// available for both. + /// + /// # Example + /// + /// ```rust + /// # struct CustomEffect(S); + /// # use rodio::{FixedSource, ConstSource}; + /// # use rodio::generators::const_source; + /// + /// // Note custom effect can only wrap a FixedSource + /// fn apply_custom_effect(source: S) -> CustomEffect { + /// CustomEffect(source) + /// } + /// + /// let source = const_source::Silence::<44100>::new(); + /// let source = source.into_fixed_source(); + /// apply_custom_effect(source); + /// ``` + fn into_fixed_source(self) -> IntoFixedSource + where + Self: Sized, + { + IntoFixedSource { inner: self } + } + + #[doc = include_str!("docs/collect_into_buffer.md")] + fn collect_into_buffer(self) -> SamplesBuffer + where + Self: Sized, + { + SamplesBuffer::new(self.collect::>()) + } + + /// Add another source to play directly after this one. + /// + /// # Example + /// + /// ```rust + /// # use rodio::const_source::ConstSource; + /// # use rodio::const_source::SamplesBuffer; + /// let preamble = SamplesBuffer::<44100, 1>::new([1.0, 1.0]); + /// let signal = SamplesBuffer::<44100, 1>::new([2.0, 2.0]); + /// + /// let mixed = preamble.chain_source(signal); + /// assert_eq!(mixed.collect::>(), vec![1.0,1.0,2.0,2.0]) + /// ``` + fn chain_source(self, next: S) -> SourceChain + where + Self: Sized, + S: ConstSource, + { + SourceChain::new(self, next) + } + + // placeholder until effects land (need this for some examples) + #[allow(missing_docs)] + fn take_duration(self, _duration: Duration) -> Placeholder + where + Self: Sized, + { + todo!() + } +} + +// placeholder until effects land (need this for some examples) +#[allow(missing_docs)] +pub struct Placeholder +where + S: ConstSource, +{ + inner: std::marker::PhantomData, +} + +/// A `DynamicSource` converted from a `ConstSource`. Useful for passing to old +/// APIs that do not accept a `ConstSource` nor `FixedSource`. +#[derive(Clone)] +pub struct IntoDynamicSource +where + S: ConstSource, +{ + inner: S, +} + +impl Iterator for IntoDynamicSource +where + S: ConstSource, +{ + type Item = Sample; + + fn next(&mut self) -> Option { + self.inner.next() + } +} + +impl DynamicSource for IntoDynamicSource +where + S: ConstSource, +{ + fn current_span_len(&self) -> Option { + None + } + + fn channels(&self) -> ChannelCount { + const { NonZeroU16::new(CH).expect("channel count must be larger then zero") } + } + + fn sample_rate(&self) -> SampleRate { + const { NonZeroU32::new(SR).expect("sample rate must be larger then zero") } + } + + fn total_duration(&self) -> Option { + self.inner.total_duration() + } +} + +/// A `FixedSource` converted from a `ConstSource`. Useful for passing to APIs +/// that do not accept a `ConstSource`. +#[derive(Clone)] +pub struct IntoFixedSource +where + S: ConstSource, +{ + inner: S, +} + +impl Iterator for IntoFixedSource +where + S: ConstSource, +{ + type Item = Sample; + + fn next(&mut self) -> Option { + self.inner.next() + } +} + +impl FixedSource for IntoFixedSource +where + S: ConstSource, +{ + fn channels(&self) -> ChannelCount { + const { NonZeroU16::new(CH).expect("channel count must be larger then zero") } + } + + fn sample_rate(&self) -> SampleRate { + const { NonZeroU32::new(SR).expect("sample rate must be larger then zero") } + } + + fn total_duration(&self) -> Option { + self.inner.total_duration() + } +} diff --git a/src/const_source/buffer.rs b/src/const_source/buffer.rs new file mode 100644 index 000000000..55aaaa6b3 --- /dev/null +++ b/src/const_source/buffer.rs @@ -0,0 +1,40 @@ +use std::sync::Arc; +use std::time::Duration; + +use crate::source::SeekError; +use crate::ConstSource; +use crate::Sample; + +/// A buffer of samples treated as a source. +#[derive(Debug, Clone)] +pub struct SamplesBuffer { + data: Arc<[Sample]>, + pos: usize, +} + +impl SamplesBuffer { + /// Builds a new `SamplesBuffer`. + /// + /// Note any call to total_duration will panic if the buffer is larger then + /// 16 billion elements. + pub fn new(data: D) -> SamplesBuffer + where + D: Into>, + { + const { assert!(SR > 0) }; + const { assert!(CH > 0) }; + + SamplesBuffer { + data: data.into().into(), + pos: 0, + } + } +} + +impl ConstSource for SamplesBuffer { + crate::common::source::buffer::source_impl! {} +} + +impl Iterator for SamplesBuffer { + crate::common::source::buffer::iter_impl! {} +} diff --git a/src/const_source/chain.rs b/src/const_source/chain.rs new file mode 100644 index 000000000..eb26c64b4 --- /dev/null +++ b/src/const_source/chain.rs @@ -0,0 +1,35 @@ +use crate::ConstSource; +use crate::Sample; + +/// Two chained sources, the one played after the other. +#[derive(Clone)] +pub struct SourceChain { + first: S1, + second: S2, + playing_inner: bool, +} + +impl, S2: ConstSource> + SourceChain +{ + pub(crate) fn new(s1: S1, s2: S2) -> Self { + SourceChain { + first: s1, + second: s2, + playing_inner: true, + } + } +} + +pub use crate::common::source::chain::ChainSeekError; +impl, S2: ConstSource> + ConstSource for SourceChain +{ + crate::common::source::chain::source_impl! {} +} + +impl, S2: ConstSource> Iterator + for SourceChain +{ + crate::common::source::chain::iter_impl! {} +} diff --git a/src/const_source/conversions.rs b/src/const_source/conversions.rs new file mode 100644 index 000000000..f648f8500 --- /dev/null +++ b/src/const_source/conversions.rs @@ -0,0 +1 @@ +pub mod channel_count; diff --git a/src/const_source/conversions/channel_count.rs b/src/const_source/conversions/channel_count.rs new file mode 100644 index 000000000..9c6ed55de --- /dev/null +++ b/src/const_source/conversions/channel_count.rs @@ -0,0 +1,78 @@ +use std::time::Duration; + +use crate::source::SeekError; +use crate::{ConstSource, Sample}; + +/// Converts between two channel counts, created using +/// [with_channel_count](ConstSource::with_channel_count) +pub struct ChannelConvertor { + input: S, + sample_repeat: Option, + next_output_sample_pos: u16, +} + +impl> + ChannelConvertor +{ + pub(crate) fn new(input: S) -> Self { + Self { + input, + sample_repeat: None, + next_output_sample_pos: 0, + } + } + + crate::common::source::add_inner_accessors! {input} +} + +impl> + ConstSource for ChannelConvertor +{ + fn total_duration(&self) -> Option { + self.input.total_duration() + } + + fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { + self.input.try_seek(pos) + } +} + +impl> Iterator + for ChannelConvertor +{ + type Item = Sample; + + fn next(&mut self) -> Option { + let result = match self.next_output_sample_pos { + 0 => { + // save first sample for mono -> stereo conversion + let value = self.input.next(); + self.sample_repeat = value; + value + } + x if x < CH_IN => { + // make sure we always end on a frame boundary + let value = self.input.next(); + assert!(value.is_some(), "Sources may not emit half frames"); + value + } + 1 => self.sample_repeat, + _ => Some(0.0), // all other added channels are empty + }; + + if result.is_some() { + self.next_output_sample_pos += 1; + } + + if self.next_output_sample_pos == CH_OUT { + self.next_output_sample_pos = 0; + + if CH_IN > CH_OUT { + for _ in CH_OUT..CH_IN { + self.input.next(); // discarding extra input + } + } + } + result + } +} diff --git a/src/conversions/channels.rs b/src/conversions/channels.rs index 94bcf4428..909f2fb43 100644 --- a/src/conversions/channels.rs +++ b/src/conversions/channels.rs @@ -1,3 +1,4 @@ +use crate::common::source::add_inner_accessors; use crate::common::ChannelCount; use crate::{Sample, Source}; use dasp_sample::Sample as _; @@ -18,13 +19,13 @@ where next_output_sample_pos: u16, } -impl ChannelCountConverter +impl ChannelCountConverter where - I: Iterator, + S: Iterator, { /// Initializes the iterator. #[inline] - pub fn new(input: I, from: ChannelCount, to: ChannelCount) -> ChannelCountConverter { + pub fn new(input: S, from: ChannelCount, to: ChannelCount) -> ChannelCountConverter { ChannelCountConverter { input, from, @@ -34,23 +35,7 @@ where } } - /// Destroys this iterator and returns the underlying iterator. - #[inline] - pub fn into_inner(self) -> I { - self.input - } - - /// Get immutable access to the underlying iterator. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Get mutable access to the underlying iterator. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } + add_inner_accessors! {input} } impl Iterator for ChannelCountConverter diff --git a/src/conversions/sample.rs b/src/conversions/sample.rs index cf7094308..0a1080214 100644 --- a/src/conversions/sample.rs +++ b/src/conversions/sample.rs @@ -11,33 +11,17 @@ pub struct SampleTypeConverter { marker: PhantomData, } -impl SampleTypeConverter { +impl SampleTypeConverter { /// Builds a new converter. #[inline] - pub fn new(input: I) -> SampleTypeConverter { + pub fn new(input: S) -> SampleTypeConverter { SampleTypeConverter { input, marker: PhantomData, } } - /// Destroys this iterator and returns the underlying iterator. - #[inline] - pub fn into_inner(self) -> I { - self.input - } - - /// Get immutable access to the underlying iterator. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Get mutable access to the underlying iterator. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for SampleTypeConverter diff --git a/src/docs/collect_into_buffer.md b/src/docs/collect_into_buffer.md new file mode 100644 index 000000000..d97b115bc --- /dev/null +++ b/src/docs/collect_into_buffer.md @@ -0,0 +1,5 @@ +Builds a new `SamplesBuffer`. + +# Panics +- Panics if the length of the buffer is larger than approximately 16 billion elements. + This is because the calculation of the duration would overflow. diff --git a/src/docs/total_duration.md b/src/docs/total_duration.md new file mode 100644 index 000000000..e05826cd6 --- /dev/null +++ b/src/docs/total_duration.md @@ -0,0 +1,6 @@ +Returns the total duration of this source, if known. + +`None` indicates at the same time "infinite" or "unknown". + +Note this value is free to change. For example if you removed an item from a +queue the total_duration lowers. diff --git a/src/docs/try_seek.md b/src/docs/try_seek.md new file mode 100644 index 000000000..cc614292b --- /dev/null +++ b/src/docs/try_seek.md @@ -0,0 +1,16 @@ +Attempts to seek to a given position in the current source. + +As long as the duration of the source is known, seek is guaranteed to saturate +at the end of the source. For example given a source that reports a total duration +of 42 seconds calling `try_seek()` with 60 seconds as argument will seek to +42 seconds. + +# Errors +This function will return [`SeekError::NotSupported`] if one of the underlying +sources does not support seeking. + +It will return an error if an implementation ran +into one during the seek. + +Seeking beyond the end of a source might return an error if the total duration of +the source is not known. diff --git a/src/fixed_source.rs b/src/fixed_source.rs index 6cc4ffee8..e0bcbaa0c 100644 --- a/src/fixed_source.rs +++ b/src/fixed_source.rs @@ -2,7 +2,16 @@ //! channel count. use std::time::Duration; -use crate::{ChannelCount, Sample, SampleRate}; +use crate::source::SeekError; +use crate::{ChannelCount, ConstSource, Sample, SampleRate}; + +mod buffer; +mod chain; +mod conversions; + +pub use buffer::SamplesBuffer; +pub use chain::SourceChain; +pub use conversions::channel_count::ChannelConverter; /// Similar to `Source`, something that can produce interleaved samples for a /// fixed amount of channels at a fixed sample rate. Those parameters never @@ -16,4 +25,215 @@ pub trait FixedSource: Iterator { /// /// `None` indicates at the same time "infinite" or "unknown". fn total_duration(&self) -> Option; + + #[allow(unused_variables)] + #[doc = include_str!("docs/try_seek.md")] + fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { + Err(SeekError::NotSupported { + underlying_source: std::any::type_name::(), + }) + } + + /// Convert from the current channel count to `channel_count`. + fn with_channel_count(self, channel_count: ChannelCount) -> ChannelConverter + where + Self: Sized, + { + ChannelConverter::new(self, channel_count) + } + + /// Tries to convert from a fixed source to a const one assuming + /// the parameters already match. If they do not this returns an error. + /// + /// If the parameters do not match you can resample using: + /// [`with_sample_rate`](Self::placeholder) and + /// [`with_channel_count`](Self::with_channel_count). + fn try_into_const_source( + self, + ) -> Result, ParameterMismatch> + where + Self: Sized, + { + if self.channels().get() != CH || self.sample_rate().get() != SR { + Err(ParameterMismatch { + sample_rate: self.sample_rate(), + channel_count: self.channels(), + }) + } else { + Ok(IntoConstSource(self)) + } + } + + /// Use this fixed source as if it's a dynamic source. You generally do not + /// want to do this since there are less effects for dynamic sources and + /// those that are available can not be implemented as efficient. This is + /// therefore purely provided for backwards compatibility. + fn into_dynamic_source(self) -> IntoDynamicSource + where + Self: Sized, + { + IntoDynamicSource(self) + } + + #[doc = include_str!("docs/collect_into_buffer.md")] + fn collect_into_buffer(self) -> SamplesBuffer + where + Self: Sized, + { + SamplesBuffer::new( + self.channels(), + self.sample_rate(), + self.collect::>(), + ) + } + + /// Add another source to play directly after this one. + /// + /// # Example + /// + /// ```rust + /// # use rodio::nz; + /// # use rodio::FixedSource; + /// # use rodio::fixed_source::SamplesBuffer; + /// # fn main() -> Result<(), Box> { + /// let preamble = SamplesBuffer::new(nz!(1), nz!(1), [1.0, 1.0]); + /// let signal = SamplesBuffer::new(nz!(1), nz!(1), [2.0, 2.0]); + /// + /// let mixed = preamble.try_chain_source(signal)?; + /// assert_eq!(mixed.collect::>(), vec![1.0,1.0,2.0,2.0]); + /// # Ok(()) + /// # } + /// ``` + fn try_chain_source( + self, + next: S, + ) -> Result, chain::ParamsMismatch> + where + Self: Sized, + { + SourceChain::new(self, next) + } + + // placeholder until effects land (need this for some examples) + #[allow(missing_docs)] + fn take_duration(self, _duration: Duration) -> Placeholder + where + Self: Sized, + { + todo!() + } + + /// here to make docs links work without the linked item being in + /// remove before next release + fn placeholder(&self) {} +} + +// placeholder until effects land (need this for some examples) +#[allow(missing_docs)] +pub struct Placeholder +where + S: FixedSource, +{ + inner: std::marker::PhantomData, +} + +impl Placeholder +where + S: FixedSource, +{ + /// placeholder + pub fn record(self) -> Placeholder { + self + } +} + +impl FixedSource for Placeholder { + fn channels(&self) -> ChannelCount { + unimplemented!("placeholder") + } + fn sample_rate(&self) -> SampleRate { + unimplemented!("placeholder") + } + fn total_duration(&self) -> Option { + unimplemented!("placeholder") + } +} + +impl Iterator for Placeholder { + type Item = Sample; + fn next(&mut self) -> Option { + unimplemented!("placeholder") + } +} + +/// A [`ConstSource`] adapted from a [`FixedSource`]. +pub struct IntoConstSource(S); + +impl ConstSource + for IntoConstSource +{ + fn total_duration(&self) -> Option { + self.0.total_duration() + } +} + +impl Iterator for IntoConstSource { + type Item = Sample; + + fn next(&mut self) -> Option { + self.0.next() + } +} + +/// Error that occurs when a [`FixedSource`] can not be converted into a +/// [`ConstSource`] with a certain sample rate and channel count. +#[derive(Debug)] +pub struct ParameterMismatch { + sample_rate: SampleRate, + channel_count: ChannelCount, +} + +impl std::error::Error for ParameterMismatch {} + +impl std::fmt::Display for ParameterMismatch { + fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result { + if self.sample_rate.get() == SR && self.channel_count.get() == CH { + unreachable!("ParameterMismatch error can only occur when params mismatch"); + } else if self.sample_rate.get() == SR && self.channel_count.get() != CH { + f.write_fmt(format_args!("Fixed source's channel count: {}, does not match target const source's channel count: {}", self.channel_count.get(), CH)) + } else if self.sample_rate.get() != SR && self.channel_count.get() != CH { + f.write_fmt(format_args!("Fixed source's sample rate and channel count ({}, {}) do not match target const source's sample rate and channel count ({} {})", self.sample_rate.get(), self.channel_count.get(), SR, CH)) + } else { + f.write_fmt(format_args!("Fixed source's sample rate : {}, does not match target const source's sample rate: {}", self.sample_rate.get(), SR)) + } + } +} + +/// A [`DynamicSource`](crate::DynamicSource) adapted from a [`FixedSource`]. +pub struct IntoDynamicSource(S); + +impl crate::DynamicSource for IntoDynamicSource { + fn current_span_len(&self) -> Option { + None + } + + fn channels(&self) -> ChannelCount { + self.0.channels() + } + + fn sample_rate(&self) -> SampleRate { + self.0.sample_rate() + } + + fn total_duration(&self) -> Option { + self.0.total_duration() + } +} + +impl Iterator for IntoDynamicSource { + type Item = crate::Sample; + + fn next(&mut self) -> Option { + self.0.next() + } } diff --git a/src/fixed_source/buffer.rs b/src/fixed_source/buffer.rs new file mode 100644 index 000000000..70618e6bd --- /dev/null +++ b/src/fixed_source/buffer.rs @@ -0,0 +1,47 @@ +use std::sync::Arc; +use std::time::Duration; + +use crate::source::SeekError; +use crate::FixedSource; +use crate::{ChannelCount, Sample, SampleRate}; + +/// A buffer of samples treated as a source. +#[derive(Debug, Clone)] +pub struct SamplesBuffer { + data: Arc<[Sample]>, + pos: usize, + channels: ChannelCount, + sample_rate: SampleRate, +} + +impl SamplesBuffer { + /// Builds a new `SamplesBuffer`. + pub fn new(channels: ChannelCount, sample_rate: SampleRate, data: D) -> SamplesBuffer + where + D: Into>, + { + let data: Arc<[Sample]> = data.into().into(); + SamplesBuffer { + data, + pos: 0, + channels, + sample_rate, + } + } +} + +impl FixedSource for SamplesBuffer { + #[inline] + fn channels(&self) -> ChannelCount { + self.channels + } + #[inline] + fn sample_rate(&self) -> SampleRate { + self.sample_rate + } + crate::common::source::buffer::source_impl! {} +} + +impl Iterator for SamplesBuffer { + crate::common::source::buffer::iter_impl! {} +} diff --git a/src/fixed_source/chain.rs b/src/fixed_source/chain.rs new file mode 100644 index 000000000..27fb5b3c1 --- /dev/null +++ b/src/fixed_source/chain.rs @@ -0,0 +1,61 @@ +use std::fmt::Display; + +use crate::FixedSource; +use crate::Sample; +use crate::{ChannelCount, SampleRate}; + +/// Two chained sources, the one played after the other. +#[derive(Clone)] +pub struct SourceChain { + first: S1, + second: S2, + playing_inner: bool, +} + +#[derive(Debug, Clone, Copy, thiserror::Error, PartialEq, Eq)] +pub struct ParamsMismatch { + sample_rate_self: SampleRate, + channel_count_self: ChannelCount, + sample_rate_adding: SampleRate, + channel_count_adding: ChannelCount, +} + +impl Display for ParamsMismatch { + fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result { + let ParamsMismatch { + sample_rate_self, + channel_count_self, + sample_rate_adding, + channel_count_adding, + } = self; + f.write_fmt(format_args!("Parameters mismatch the source chained onto Self does not have the same parameters. Self has sample rate: {sample_rate_self} and channel count: {channel_count_self} while the source being chained has sample rate: {sample_rate_adding} and channel count: {channel_count_adding}.")) + } +} + +impl SourceChain { + pub(crate) fn new(s1: S1, s2: S2) -> Result { + if s1.sample_rate() != s2.sample_rate() || s1.channels() != s2.channels() { + Err(ParamsMismatch { + sample_rate_self: s1.sample_rate(), + channel_count_self: s1.channels(), + sample_rate_adding: s2.sample_rate(), + channel_count_adding: s2.channels(), + }) + } else { + Ok(SourceChain { + first: s1, + second: s2, + playing_inner: true, + }) + } + } +} + +pub use crate::common::source::chain::ChainSeekError; +impl FixedSource for SourceChain { + crate::common::source::chain::source_impl! {} +} + +impl Iterator for SourceChain { + crate::common::source::chain::iter_impl! {} +} diff --git a/src/fixed_source/conversions.rs b/src/fixed_source/conversions.rs new file mode 100644 index 000000000..f648f8500 --- /dev/null +++ b/src/fixed_source/conversions.rs @@ -0,0 +1 @@ +pub mod channel_count; diff --git a/src/fixed_source/conversions/channel_count.rs b/src/fixed_source/conversions/channel_count.rs new file mode 100644 index 000000000..0c6070489 --- /dev/null +++ b/src/fixed_source/conversions/channel_count.rs @@ -0,0 +1,84 @@ +use std::time::Duration; + +use crate::source::SeekError; +use crate::{ChannelCount, Sample}; +use crate::{FixedSource, SampleRate}; + +/// Converts between two channel counts, created using +/// [with_channel_count](FixedSource::with_channel_count) +pub struct ChannelConverter { + input: S, + pub(crate) target: ChannelCount, + sample_repeat: Option, + next_output_sample_pos: u16, +} + +impl ChannelConverter { + pub(crate) fn new(input: S, target: ChannelCount) -> Self { + Self { + input, + target, + sample_repeat: None, + next_output_sample_pos: 0, + } + } + + crate::common::source::add_inner_accessors! {input} +} + +impl FixedSource for ChannelConverter { + fn channels(&self) -> ChannelCount { + self.target + } + + fn sample_rate(&self) -> SampleRate { + self.input.sample_rate() + } + + fn total_duration(&self) -> Option { + self.input.total_duration() + } + + fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { + self.input.try_seek(pos) + } +} + +// TODO optimize (still assumes dynamicsource) +impl Iterator for ChannelConverter { + type Item = crate::Sample; + + fn next(&mut self) -> Option { + let result = match self.next_output_sample_pos { + 0 => { + // save first sample for mono -> stereo conversion + let value = self.input.next(); + self.sample_repeat = value; + value + } + x if x < self.input.channels().get() => { + // make sure we always end on a frame boundary + let value = self.input.next(); + assert!(value.is_some(), "Sources may not emit half frames"); + value + } + 1 => self.sample_repeat, + _ => Some(0.0), // all other added channels are empty + }; + + if result.is_some() { + self.next_output_sample_pos += 1; + } + + if self.next_output_sample_pos == self.target.get() { + self.next_output_sample_pos = 0; + + if self.input.channels() > self.target { + for _ in self.target.get()..self.input.channels().get() { + self.input.next(); // discarding extra input + } + } + } + result + } +} diff --git a/src/generators.rs b/src/generators.rs new file mode 100644 index 000000000..c0fed4a50 --- /dev/null +++ b/src/generators.rs @@ -0,0 +1,15 @@ +//! Sources that produce sound without consuming anything + +// Note we make everything be it effects or generators available under / + +mod silence; + +/// Generators with both the sample rate and channel count known at compile time. +pub mod const_source { + pub use super::silence::const_source::Silence; +} + +/// Generators where the sample rate and channel may be passed in at runtime +pub mod fixed_source { + pub use super::silence::fixed_source::Silence; +} diff --git a/src/generators/silence.rs b/src/generators/silence.rs new file mode 100644 index 000000000..baa7516f8 --- /dev/null +++ b/src/generators/silence.rs @@ -0,0 +1,134 @@ +pub mod fixed_source { + use std::time::Duration; + + use crate::source::SeekError; + use crate::{nz, FixedSource}; + use crate::{ChannelCount, SampleRate}; + + /// A source producing an infinite amount of Silence. Like all generators you + /// probably want to limit the duration of this source. + /// + /// # Example + /// Padding a [`TakeDuration`](crate::fixed_source::Placeholder) to + /// guarantee an exact playtime: + /// + /// ```rust,no_run + /// # use std::time::Duration; + /// # use rodio::nz; + /// # use rodio::generators::fixed_source::Silence; + /// # use rodio::{ConstSource, FixedSource}; + /// # fn main() -> Result<(), Box> { + /// # let unknown_length = Silence::new(nz!(44100)); + /// let silence = Silence::new(nz!(44100)); + /// let two_seconds = unknown_length + /// .try_chain_source(silence)? + /// .take_duration(Duration::from_secs(2)); + /// # Ok(()) + /// # } + /// ``` + pub struct Silence { + sample_rate: SampleRate, + } + + impl Silence { + /// Create an infinite silence + pub fn new(sample_rate: SampleRate) -> Self { + Self { sample_rate } + } + } + + impl FixedSource for Silence { + fn channels(&self) -> ChannelCount { + nz!(1) + } + + fn sample_rate(&self) -> SampleRate { + self.sample_rate + } + + fn total_duration(&self) -> Option { + None + } + + /// This does nothing since all silence is equal :3 + fn try_seek(&mut self, _: Duration) -> Result<(), SeekError> { + Ok(()) + } + } + + impl Iterator for Silence { + type Item = crate::Sample; + + fn next(&mut self) -> Option { + Some(0.0) + } + + fn size_hint(&self) -> (usize, Option) { + (usize::MAX, None) + } + } +} + +pub mod const_source { + use std::time::Duration; + + use crate::source::SeekError; + use crate::ConstSource; + + /// A source producing an infinite amount of Silence. Like all generators you + /// probably want to limit the duration of this source. + /// + /// # Example + /// Padding a [`TakeDuration`](crate::const_source::Placeholder) to + /// guarantee an exact playtime: + /// + /// ```rust,no_run + /// # use std::time::Duration; + /// # use rodio::nz; + /// # use rodio::generators::const_source::Silence; + /// # use rodio::ConstSource; + /// # let unknown_length = Silence::new(); + /// + /// let silence: Silence<44100> = Silence::new(); + /// let two_seconds = unknown_length + /// .chain_source(silence) + /// .take_duration(Duration::from_secs(2)); + /// ``` + pub struct Silence; + + impl Silence { + /// Create an infinite silence + pub fn new() -> Self { + Self + } + } + + impl Default for Silence { + fn default() -> Self { + Self + } + } + + impl ConstSource for Silence { + fn total_duration(&self) -> Option { + None + } + + /// This does nothing since all silence is equal :3 + fn try_seek(&mut self, _: Duration) -> Result<(), SeekError> { + Ok(()) + } + } + + impl Iterator for Silence { + type Item = crate::Sample; + + fn next(&mut self) -> Option { + Some(0.0) + } + + fn size_hint(&self) -> (usize, Option) { + (usize::MAX, None) + } + } +} diff --git a/src/lib.rs b/src/lib.rs index dbd6174c3..034912b87 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -226,8 +226,12 @@ mod wav_output; pub mod buffer; pub mod conversions; pub mod decoder; -#[cfg(feature = "experimental")] + +pub mod const_source; pub mod fixed_source; + +pub mod generators; + pub mod math; #[cfg(feature = "recording")] /// Microphone input support for audio recording. @@ -238,8 +242,8 @@ pub mod source; pub mod static_buffer; pub use crate::common::{BitDepth, ChannelCount, Float, Sample, SampleRate, DEFAULT_SAMPLE_RATE}; +pub use crate::const_source::ConstSource; pub use crate::decoder::Decoder; -#[cfg(feature = "experimental")] pub use crate::fixed_source::FixedSource; pub use crate::player::Player; pub use crate::source::Source; @@ -250,3 +254,5 @@ pub use crate::stream::{play, DeviceSinkBuilder, DeviceSinkError, MixerDeviceSin pub use crate::wav_output::wav_to_file; #[cfg(feature = "wav_output")] pub use crate::wav_output::wav_to_writer; + +pub use Source as DynamicSource; diff --git a/src/microphone.rs b/src/microphone.rs index 77b9b4839..c1b3b2502 100644 --- a/src/microphone.rs +++ b/src/microphone.rs @@ -4,7 +4,7 @@ //! //! ```no_run //! use rodio::microphone::MicrophoneBuilder; -//! use rodio::Source; +//! use rodio::FixedSource; //! use std::time::Duration; //! //! # fn main() -> Result<(), Box> { @@ -106,7 +106,7 @@ use std::time::Duration; use crate::common::assert_error_traits; use crate::conversions::SampleTypeConverter; -use crate::{Sample, Source}; +use crate::Sample; mod builder; mod config; @@ -188,25 +188,6 @@ pub struct Microphone { config: InputConfig, } -impl Source for Microphone { - fn current_span_len(&self) -> Option { - None - } - - fn channels(&self) -> crate::ChannelCount { - self.config.channel_count - } - - fn sample_rate(&self) -> crate::SampleRate { - self.config.sample_rate - } - - fn total_duration(&self) -> Option { - None - } -} - -#[cfg(feature = "experimental")] impl crate::FixedSource for Microphone { fn channels(&self) -> crate::ChannelCount { self.config.channel_count diff --git a/src/microphone/builder.rs b/src/microphone/builder.rs index a698424bd..69afe1646 100644 --- a/src/microphone/builder.rs +++ b/src/microphone/builder.rs @@ -538,7 +538,7 @@ where /// # Example /// ```no_run /// # use rodio::microphone::MicrophoneBuilder; - /// # use rodio::Source; + /// # use rodio::FixedSource; /// # use std::time::Duration; /// let mic = MicrophoneBuilder::new() /// .default_device()? diff --git a/src/source/amplify.rs b/src/source/amplify.rs index c9dcec486..fb8ab8b47 100644 --- a/src/source/amplify.rs +++ b/src/source/amplify.rs @@ -21,7 +21,7 @@ pub struct Amplify { factor: Float, } -impl Amplify { +impl Amplify { /// Modifies the amplification factor. #[inline] pub fn set_factor(&mut self, factor: Float) { @@ -34,23 +34,7 @@ impl Amplify { self.factor = math::db_to_linear(factor); } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Amplify diff --git a/src/source/automatic_gain_control/agc.rs b/src/source/automatic_gain_control/agc.rs index 2afec0202..54f4db8ac 100644 --- a/src/source/automatic_gain_control/agc.rs +++ b/src/source/automatic_gain_control/agc.rs @@ -112,9 +112,9 @@ pub struct AutomaticGainControl { slow_down_state: SlowDownState, } -impl AutomaticGainControl +impl AutomaticGainControl where - I: Source, + S: Source, { /// Constructs an `AutomaticGainControl` object with specified parameters. /// @@ -129,16 +129,16 @@ where /// `floor` - The minimum output level (gain floor) that the AGC will not go below #[inline] pub(crate) fn new( - input: I, + input: S, target_level: Float, attack_time: Duration, release_time: Duration, absolute_max_gain: Float, peak_tracking_window: Duration, floor: Float, - ) -> AutomaticGainControl + ) -> AutomaticGainControl where - I: Source, + S: Source, { let sample_rate = input.sample_rate(); let attack_duration = duration_to_float(attack_time); @@ -377,7 +377,7 @@ where } #[inline] - fn process_sample(&mut self, sample: I::Item) -> I::Item { + fn process_sample(&mut self, sample: S::Item) -> S::Item { // Cache atomic loads at the start - avoids repeated atomic operations let target_level = self.target_level(); let absolute_max_gain = self.absolute_max_gain(); @@ -468,15 +468,7 @@ where sample * self.current_gain } - /// Returns an immutable reference to the inner source. - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for AutomaticGainControl diff --git a/src/source/channel_volume.rs b/src/source/channel_volume.rs index 29d5a59ba..97367ccf6 100644 --- a/src/source/channel_volume.rs +++ b/src/source/channel_volume.rs @@ -19,14 +19,14 @@ where current_sample: Option, } -impl ChannelVolume +impl ChannelVolume where - I: Source, + S: Source, { /// Wrap the input source and make it mono. Play that mono sound to each /// channel at the volume set by the user. The volume can be changed using /// [`ChannelVolume::set_volume`]. - pub fn new(input: I, channel_volumes: Vec) -> ChannelVolume { + pub fn new(input: S, channel_volumes: Vec) -> ChannelVolume { let channel_count = channel_volumes.len(); // See next() implementation. ChannelVolume { input, @@ -42,28 +42,12 @@ where self.channel_volumes[channel] = volume; } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } - /// Gets the volume for a given channel number. Returns `None` if channel number is invalid. pub fn volume(&self, channel: usize) -> Option { self.channel_volumes.get(channel).copied() } + + crate::common::source::add_inner_accessors! {input} } impl Iterator for ChannelVolume diff --git a/src/source/delay.rs b/src/source/delay.rs index 5c835dc55..25d37f3d2 100644 --- a/src/source/delay.rs +++ b/src/source/delay.rs @@ -35,27 +35,11 @@ pub struct Delay { requested_duration: Duration, } -impl Delay +impl Delay where - I: Source, + S: Source, { - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Delay diff --git a/src/source/distortion.rs b/src/source/distortion.rs index 381e9a0df..f01aa68ec 100644 --- a/src/source/distortion.rs +++ b/src/source/distortion.rs @@ -24,7 +24,7 @@ pub struct Distortion { threshold: Float, } -impl Distortion { +impl Distortion { /// Modifies the distortion gain. #[inline] pub fn set_gain(&mut self, gain: Float) { @@ -37,23 +37,7 @@ impl Distortion { self.threshold = threshold; } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Distortion diff --git a/src/source/done.rs b/src/source/done.rs index 64a51a500..80f1f70e2 100644 --- a/src/source/done.rs +++ b/src/source/done.rs @@ -30,24 +30,10 @@ where signal_sent: false, } } +} - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } +impl Done { + crate::common::source::add_inner_accessors! {input} } impl Iterator for Done diff --git a/src/source/from_iter.rs b/src/source/from_iter.rs index 0ade85bc6..1801cbee7 100644 --- a/src/source/from_iter.rs +++ b/src/source/from_iter.rs @@ -40,10 +40,10 @@ pub struct FromIter { sample_rate: SampleRate, } -impl FromIter { +impl FromIter { /// Creates a new `FromIter` from an iterator and audio parameters. #[inline] - pub fn new(iter: I, channels: ChannelCount, sample_rate: SampleRate) -> Self { + pub fn new(iter: S, channels: ChannelCount, sample_rate: SampleRate) -> Self { Self { iter, channels, @@ -51,23 +51,7 @@ impl FromIter { } } - /// Destroys this source and returns the underlying iterator. - #[inline] - pub fn into_inner(self) -> I { - self.iter - } - - /// Get immutable access to the underlying iterator. - #[inline] - pub fn inner(&self) -> &I { - &self.iter - } - - /// Get mutable access to the underlying iterator. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.iter - } + crate::common::source::add_inner_accessors! {iter} } impl Iterator for FromIter diff --git a/src/source/linear_ramp.rs b/src/source/linear_ramp.rs index 4467e730d..68d65f6e3 100644 --- a/src/source/linear_ramp.rs +++ b/src/source/linear_ramp.rs @@ -46,27 +46,11 @@ pub struct LinearGainRamp { span: SpanTracker, } -impl LinearGainRamp +impl LinearGainRamp where - I: Source, + S: Source, { - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for LinearGainRamp diff --git a/src/source/mod.rs b/src/source/mod.rs index b8ea8553a..237735bee 100644 --- a/src/source/mod.rs +++ b/src/source/mod.rs @@ -217,9 +217,7 @@ pub trait Source: Iterator { /// Returns the rate at which the source should be played. In number of samples per second. fn sample_rate(&self) -> SampleRate; - /// Returns the total duration of this source, if known. - /// - /// `None` indicates at the same time "infinite" or "unknown". + #[doc = include_str!("../docs/total_duration.md")] fn total_duration(&self) -> Option; /// Stores the source in a buffer in addition to returning it. This iterator can be cloned. @@ -697,23 +695,8 @@ pub trait Source: Iterator { SampleRateConverter::new(self, target_rate, config) } - /// Attempts to seek to a given position in the current source. - /// - /// As long as the duration of the source is known, seek is guaranteed to saturate - /// at the end of the source. For example given a source that reports a total duration - /// of 42 seconds calling `try_seek()` with 60 seconds as argument will seek to - /// 42 seconds. - /// - /// # Errors - /// This function will return [`SeekError::NotSupported`] if one of the underlying - /// sources does not support seeking. - /// - /// It will return an error if an implementation ran - /// into one during the seek. - /// - /// Seeking beyond the end of a source might return an error if the total duration of - /// the source is not known. #[allow(unused_variables)] + #[doc = include_str!("../docs/try_seek.md")] fn try_seek(&mut self, pos: Duration) -> Result<(), SeekError> { Err(SeekError::NotSupported { underlying_source: std::any::type_name::(), diff --git a/src/source/pausable.rs b/src/source/pausable.rs index 1b05ab0d7..32fd2c720 100644 --- a/src/source/pausable.rs +++ b/src/source/pausable.rs @@ -34,9 +34,9 @@ pub struct Pausable { remaining_paused_samples: u16, } -impl Pausable +impl Pausable where - I: Source, + S: Source, { /// Sets whether the filter applies. /// @@ -56,23 +56,7 @@ where self.paused_channels.is_some() } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Pausable diff --git a/src/source/periodic.rs b/src/source/periodic.rs index b251ea316..a6ba5317f 100644 --- a/src/source/periodic.rs +++ b/src/source/periodic.rs @@ -41,29 +41,13 @@ pub struct PeriodicAccess { samples_until_update: usize, } -impl PeriodicAccess +impl PeriodicAccess where - I: Source, + S: Source, - F: FnMut(&mut I), + F: FnMut(&mut S), { - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for PeriodicAccess diff --git a/src/source/position.rs b/src/source/position.rs index f41848eb6..5b85f9539 100644 --- a/src/source/position.rs +++ b/src/source/position.rs @@ -28,24 +28,8 @@ pub struct TrackPosition { span: SpanTracker, } -impl TrackPosition { - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } +impl TrackPosition { + crate::common::source::add_inner_accessors! {input} } impl TrackPosition diff --git a/src/source/skip.rs b/src/source/skip.rs index e1656aee0..900a164e4 100644 --- a/src/source/skip.rs +++ b/src/source/skip.rs @@ -89,27 +89,11 @@ pub struct SkipDuration { skipped_duration: Duration, } -impl SkipDuration +impl SkipDuration where - I: Source, + S: Source, { - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for SkipDuration diff --git a/src/source/skippable.rs b/src/source/skippable.rs index bfa973998..73eb73358 100644 --- a/src/source/skippable.rs +++ b/src/source/skippable.rs @@ -23,7 +23,7 @@ pub struct Skippable { do_skip: bool, } -impl Skippable { +impl Skippable { /// Skips the current source #[inline] pub fn skip(&mut self) { @@ -36,23 +36,7 @@ impl Skippable { self.do_skip } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Skippable diff --git a/src/source/speed.rs b/src/source/speed.rs index 5d599a132..a9bcb5dea 100644 --- a/src/source/speed.rs +++ b/src/source/speed.rs @@ -64,9 +64,9 @@ pub struct Speed { factor: f32, } -impl Speed +impl Speed where - I: Source, + S: Source, { /// Modifies the speed factor. #[inline] @@ -74,23 +74,7 @@ where self.factor = factor; } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Speed diff --git a/src/source/stoppable.rs b/src/source/stoppable.rs index f89e0017c..12c412e90 100644 --- a/src/source/stoppable.rs +++ b/src/source/stoppable.rs @@ -19,30 +19,14 @@ pub struct Stoppable { stopped: bool, } -impl Stoppable { +impl Stoppable { /// Stops the sound. #[inline] pub fn stop(&mut self) { self.stopped = true; } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} } impl Iterator for Stoppable diff --git a/src/source/take.rs b/src/source/take.rs index fb6ef2864..631a2f4f2 100644 --- a/src/source/take.rs +++ b/src/source/take.rs @@ -56,35 +56,19 @@ pub struct TakeDuration { silence_samples_remaining: usize, } -impl TakeDuration +impl TakeDuration where - I: Source, + S: Source, { /// Returns the duration elapsed for each sample extracted. #[inline] - fn get_duration_per_sample(input: &I) -> Duration { + fn get_duration_per_sample(input: &S) -> Duration { let ns = NANOS_PER_SEC / (input.sample_rate().get() as u64 * input.channels().get() as u64); // \|/ the maximum value of `ns` is one billion, so this can't fail Duration::new(0, ns as u32) } - /// Returns a reference to the inner source. - #[inline] - pub fn inner(&self) -> &I { - &self.input - } - - /// Returns a mutable reference to the inner source. - #[inline] - pub fn inner_mut(&mut self) -> &mut I { - &mut self.input - } - - /// Returns the inner source. - #[inline] - pub fn into_inner(self) -> I { - self.input - } + crate::common::source::add_inner_accessors! {input} /// Make the truncated source end with a FadeOut. The fadeout covers the /// entire length of the take source.