[PR #117] [MERGED] fix: initialize vorbis stream on the first read #219

New issue

Closed

opened 2026-02-28 14:26:17 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:26:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/devgianlu/go-librespot/pull/117
Author: @aykevl
Created: 10/6/2024
Status: ✅ Merged
Merged: 10/6/2024
Merged by: @devgianlu

Base: master ← Head: fix-play-start-stutter-2

📝 Commits (1)

ee50dff fix: initialize vorbis stream on the first read

📊 Changes

1 file changed (+6 additions, -2 deletions)

View changed files

📝 vorbis/decoder.go (+6 -2)

📄 Description

This fixes a stutter that can be heard on some songs, where it would initialize the stream, seek to a different position (or 0), and then continue. This is most audible on tracks that immediately start playing audio without fade-in.

This is probably a better fix than #116.

fixes #115

_{🔄 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/devgianlu/go-librespot/pull/117 **Author:** [@aykevl](https://github.com/aykevl) **Created:** 10/6/2024 **Status:** ✅ Merged **Merged:** 10/6/2024 **Merged by:** [@devgianlu](https://github.com/devgianlu) **Base:** `master` ← **Head:** `fix-play-start-stutter-2` --- ### 📝 Commits (1) - [`ee50dff`](https://github.com/devgianlu/go-librespot/commit/ee50dffa3037519d3dd8c67ebed10653c4706307) fix: initialize vorbis stream on the first read ### 📊 Changes **1 file changed** (+6 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `vorbis/decoder.go` (+6 -2) </details> ### 📄 Description This fixes a stutter that can be heard on some songs, where it would initialize the stream, seek to a different position (or 0), and then continue. This is most audible on tracks that immediately start playing audio without fade-in. This is probably a better fix than #116. fixes #115 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>