mirror of
https://github.com/Rigellute/spotify-tui.git
synced 2026-04-27 00:25:53 +03:00
[PR #124] [MERGED] Fix panic when seeking beyond track boundaries #618
Labels
No labels
bug
enhancement
good first issue
help wanted
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-tui#618
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/Rigellute/spotify-tui/pull/124
Author: @sputnick1124
Created: 11/1/2019
Status: ✅ Merged
Merged: 11/1/2019
Merged by: @Rigellute
Base:
master← Head:123-seek-panic📝 Commits (2)
50f2bf6fix: prevent panic on impossible backward seek827e041fix: prevent panic when seeking past end of track📊 Changes
1 file changed (+15 additions, -2 deletions)
View changed files
📝
src/app.rs(+15 -2)📄 Description
Addresses https://github.com/Rigellute/spotify-tui/issues/123.
Not sure if there is an edge case to be concerned about:
If the user is at the end of a track and attempts to seek forward twice quickly (between ticks), it may be possible that
self.next_trackis called twice and playback unexpectedly skips forward two tracks instead of the one as expected.Am open to suggestions for better alternatives.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.