PDA

View Full Version : Live365 Patches



chrisp810
2005-07-16, 11:37
Attached is a svn diff for a few Live365 plugin patches I've made over the past few weeks.

Fixes include:
- Proper error reporting when your Live365 login times out or has been superceded by a login elsewhere
- No more crash when listening to a station with only one song in the playlist
- Better login handling to avoid logging in multiple times if more than one player is browsing/playing Live365 stations at once.

All have been tested pretty thoroughly with every scenario I can think of, but as usual, any further testing is welcome.

Chris

vidurapparao
2005-07-16, 17:12
Thanks, Chris. Will review, test and commit.

--Vidur

chrisp810 wrote:

>Attached is a svn diff for a few Live365 plugin patches I've made over
>the past few weeks.
>
>Fixes include:
>- Proper error reporting when your Live365 login times out or has
>been superceded by a login elsewhere
>- No more crash when listening to a station with only one song in
>the playlist
>- Better login handling to avoid logging in multiple times if more
>than one player is browsing/playing Live365 stations at once.
>
>All have been tested pretty thoroughly with every scenario I can think
>of, but as usual, any further testing is welcome.
>
>Chris
>
>
>+-------------------------------------------------------------------+
>|Filename: Live365diff.txt |
>|Download: http://forums.slimdevices.com/attachment.php?attachmentid=188|
>+-------------------------------------------------------------------+
>
>
>

vidurapparao
2005-07-18, 16:49
Thanks again, Chris. The patch went in today.

While reviewing and testing, I noticed that there's still some global
state that should be per-client ($mainModeIdex, @mainModeItems,
$searchModeIdx). Since this needs to be fixed for SqueezeNetwork, I'm
going to add it to my list. But, if you think you might be able to work
on it in the next couple of days, I'd welcome the help and would
backport the change to SqueezeNetwork.

--Vidur

chrisp810
2005-07-19, 09:34
While reviewing and testing, I noticed that there's still some global
state that should be per-client ($mainModeIdex, @mainModeItems,
$searchModeIdx). Since this needs to be fixed for SqueezeNetwork, I'm
going to add it to my list. But, if you think you might be able to work
on it in the next couple of days, I'd welcome the help and would
backport the change to SqueezeNetwork.

Thanks for running it through its paces. I should be able to check the other global info and throw something together by the weekend.

Chris

pcumming
2005-08-26, 15:33
Thanks, Peter
pcumming@yahoo.com

808state
2005-11-12, 19:08
bump..also, what version of this patch was it incorporated into slimserver?

mherger
2005-11-13, 03:19
> bump..also, what version of this patch was it incorporated into
> slimserver?

If you read the thread you'll see that it had been integrated in the main
code 07-19. No need to apply it manually.

--

Michael

-----------------------------------------------------------
Help translate SlimServer by using the
StringEditor Plugin (http://www.herger.net/slim/)