Bugzilla – Bug 1192
playlist errors when starting server
Last modified: 2008-12-15 11:57:24 UTC
seen in r2720: Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in string eq at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238. Use of uninitialized value in numeric eq (==) at /Users/kpearsall/6/server/Slim/Buttons/Playlist.pm line 238.
Looks like you were here last.
Created attachment 375 [details] parens help this clears the warnings, but it is odd that mode isn't defined at any point. that would be something fairly recent. that function was expanded recently, but it still basically the same meaning as it was.
nice and quiet with that patch. :-)
Created attachment 378 [details] Another patch that fixes some of the causes Heh. We overlapped on this one. I've attached a patch that fixes at least one of the causes of the warning in the first place - calling reconnect() before init() when we first see a client. This is all triggered by our over-aggressive calls to set the visualizer on the player. We may want to wait for Dean to fix some of this (and he is working on it) before we go with either of these.
According to vidur, my last checkin fixed this, so it's in last night's nightly build. Please confirm.
yes. looks good. thank you!
This bug has been fixed in the latest release of SqueezeCenter! Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already. If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.