kwarklabs

Same error observed on LGE Nexus 5X running Android 8.1.0 when using startActivity(), so the problem isn't limited to when using navController.

@philippe44 thanks for the comment. The changes in this PR are focused on problems with remote server stream and are in different subroutines from those that handle local player reconnect....

@philippe44 noted. With a remote stream playing, we've tested player power off/on and player restart within the 5 minute forgetClient timer period and after the forgetClient timer elapsed. We tested...

@philippe44 Thanks. Have responded to your inline comments and reverted code in the spots you hightlighted.

@michaelherger Side effects are only likely in streaming failure scenarios, noting that normal end-of-file events can be considered as failure scenarios in the context of the StreamingController, beside stream corruptions...

Thanks for feedback @michaelherger. We'll have a think about why that might be happening with Spotty on Pi OS. Which player where you using with Pi?

@michaelherger Because you said the last seconds are repeated 2-4x before jumping the problem you are seeing is likely on line 1429 in the _willRetry subroutine. Suspect that either or...

@philippe44 Thanks for confirming. We have added checks for undefined elapsed and duration at line 1429 to the PR accordingly. @philippe44 any idea which players are affected? Can't immediately see...

Thanks for testing @michaelherger and for the attachments. We've reviewed the video and the logfile and made two changes to the PR as a result: 1. The same undef checks...

Thanks for this @michaelherger. Your initial seek can be seen at lines 82 and 141: Line 82 [22-05-08 10:22:14.9267] Slim::Player::Song::open (370) Adding seekdata { timeOffset => "170.804928131417" } Line 141...