Home of the Squeezebox™ & Transporter® network music players.
Results 1 to 2 of 2
  1. #1
    Senior Member gharris999's Avatar
    Join Date
    Apr 2005
    Location
    Santa Fe, NM
    Posts
    3,299

    Developer humor..

    Not often that code makes me laugh. But the latest 7.8 check-in by felix did make me chuckle.

    Code:
    Author: fmueller
    Date: Fri Mar 16 08:49:49 2012
    New Revision: 33895
    
    URL: http://svn.slimdevices.com/slim?rev=33895&view=rev
    Log:
    Bug: n/a 
    Description: Alarm - use more robust test about whether a stream is playing.
    ...
    --- 7.8/trunk/server/Slim/Utils/Alarm.pm (original)
    +++ 7.8/trunk/server/Slim/Utils/Alarm.pm Fri Mar 16 08:49:49 2012
    ...
    -	if (!$client->isPlaying) {
    +	if (!$client->isPlaying('really')) {
    I like this approach to coding. If at first you don't succeed, just express yourself more forcefully.

  2. #2
    Senior Member
    Join Date
    Apr 2005
    Location
    Colorado
    Posts
    10,073
    Not just playing. Really playing.

    I wonder if this attempts to address the age-old Squeezebox problem of the server thinking that it's streaming music to a dead silent player.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •