Aesculus wrote: > Do I need to do something with the defaults here? Is it the fact that if > there is no number it errors or does it not like 0?
I have identified some timing problems that are more likely to occur when the "power on" and "power off" delays are zero and have been testing some extensive changes to address them, including forcing a minimum 1-second delay for both. Of course, the menu could also be modified to change the minimum and default values but I didn't want to get into that. In David's case, my guess is that a "power off" sequence did not complete successfully, which results in a socket connection being left open and subsequent errors. I also believe that the default timeout value for socket commands should be increased from 0.125 to 0.25 seconds, as the Denon protocol doc states that a response should be received within 0.2 seconds. Ideally, there could be a menu option for this, with a default value of 0.25 and a range of 0.25 to 1 sec. The doc also states that a client app should wait at least 1 second after a power-on (PWON) command before issuing the next command. In our case, this would be the delay before the quick-select (MSQUICK) command, which currently defaults to 1 sec. I have also identified and fixed a problem where the volume was not being synched during the "power on" process. The code was there to do it but the command failed every time with a comm error because the socket was closed prematurely. With this fix, the only time the player and AVR volumes should ever get out of synch is when the AVR volume is changed outside the Squeezebox domain. In fact, I have removed the code to synch volume on a "pause" as it is no longer necessary. Oh, and the plugin will no longer ignore the first "power on" command after LMS is started. 😉 As I mentioned, I'm still testing all this right now but would like to get some feedback after I feel that it is ready for "beta". Sam ------------------------------------------------------------------------ SamY's Profile: http://forums.slimdevices.com/member.php?userid=63495 View this thread: http://forums.slimdevices.com/showthread.php?t=60114 _______________________________________________ plugins mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/plugins
