[PR #114] [CLOSED] Volume normalization #785

Closed
opened 2026-02-27 19:59:57 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/librespot-org/librespot/pull/114
Author: @herrernst
Created: 1/31/2018
Status: Closed

Base: masterHead: volume-normalization


📝 Commits (7)

  • 0817b08 apply pre-computed track gain from Spotify's proprietary ogg header
  • f5989f4 enable normalization only if flag is passed
  • 2eabb93 log other gain info
  • 38f18e3 warn on clipping
  • 67765f7 prevent clipping; add option to apply pre-gain
  • 89df82b Merge branch 'master' into volume-normalization
  • d309ddd Accidentally removed a ;

📊 Changes

3 files changed (+93 additions, -9 deletions)

View changed files

📝 core/src/config.rs (+6 -0)
📝 src/main.rs (+6 -0)
📝 src/player.rs (+81 -9)

📄 Description

See #15


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/librespot-org/librespot/pull/114 **Author:** [@herrernst](https://github.com/herrernst) **Created:** 1/31/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `volume-normalization` --- ### 📝 Commits (7) - [`0817b08`](https://github.com/librespot-org/librespot/commit/0817b08507066f448779818ea8ec214958b1d4d9) apply pre-computed track gain from Spotify's proprietary ogg header - [`f5989f4`](https://github.com/librespot-org/librespot/commit/f5989f4362742e471768d911218f113bf40d8181) enable normalization only if flag is passed - [`2eabb93`](https://github.com/librespot-org/librespot/commit/2eabb930e0a8ee1e02dd62c46780fb33a7a19521) log other gain info - [`38f18e3`](https://github.com/librespot-org/librespot/commit/38f18e3adb523f18fc390485fe03dea72da6adb0) warn on clipping - [`67765f7`](https://github.com/librespot-org/librespot/commit/67765f747863ce63485a53971ec0e30b8a8ba96c) prevent clipping; add option to apply pre-gain - [`89df82b`](https://github.com/librespot-org/librespot/commit/89df82bd032cc00a35e6f03736f9153b93a1fa1c) Merge branch 'master' into volume-normalization - [`d309ddd`](https://github.com/librespot-org/librespot/commit/d309ddd289181e1b071e4aceb921ab34301faa4c) Accidentally removed a `;` ### 📊 Changes **3 files changed** (+93 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `core/src/config.rs` (+6 -0) 📝 `src/main.rs` (+6 -0) 📝 `src/player.rs` (+81 -9) </details> ### 📄 Description See #15 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 19:59:57 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/librespot#785
No description provided.