Skip to content

Switch application to F32le samples, replace sample_log with wav consumer #24

Merged
hal7df merged 4 commits into
mainfrom
issues/14
Aug 6, 2023
Merged

Switch application to F32le samples, replace sample_log with wav consumer #24
hal7df merged 4 commits into
mainfrom
issues/14

Conversation

@hal7df
Copy link
Copy Markdown
Owner

@hal7df hal7df commented Aug 6, 2023

  • Changes the sample format used internally by the application to F32le, to match the sample format used internally by songbird.
  • Moves the discord module into a new top-level consumer module.
  • Adds a wav consumer, which writes intercepted audio into a wav file. This consumer is available for production builds, but requires the wav feature to be enabled (which it is by default). This replaces the previous sample_log consumer.
  • Adds configuration and validation for the new wav consumer.

Closes #14.

hal7df added 3 commits July 30, 2023 22:51
- Move discord and sample_log modules into the consumer module.
- Print the warning when -o is ignored using eprintln! rather than warn!
  as it is called before the logging system is initialized.
- Fix incorrect comparison when determining whether to show warning for
  max wav recording length.
@hal7df hal7df added this to the 0.2.0 milestone Aug 6, 2023
@hal7df hal7df merged commit 6e5e4fe into main Aug 6, 2023
@hal7df hal7df deleted the issues/14 branch August 6, 2023 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate switching to f32 audio samples

1 participant