Author: titmuss
Date: Sat Jan 19 12:49:20 2008
New Revision: 1512
URL: http://svn.slimdevices.com?rev=1512&root=Jive&view=rev
Log:
Bug: N/A
Description:
Fix AppletManger, applets should always be freed unless they return false. The
SetupSqueezebox applets was returning nil and that caused problems. Warn if
applets
return nil on free.
were returning nil, and that caused init
Modified:
trunk/jive/src/pkg/jive/share/jive/AppletManager.lua
Modified: trunk/jive/src/pkg/jive/share/jive/AppletManager.lua
URL:
http://svn.slimdevices.com/trunk/jive/src/pkg/jive/share/jive/AppletManager.lua?rev=1512&root=Jive&r1=1511&r2=1512&view=diff
==============================================================================
--- trunk/jive/src/pkg/jive/share/jive/AppletManager.lua (original)
+++ trunk/jive/src/pkg/jive/share/jive/AppletManager.lua Sat Jan 19 12:49:20
2008
@@ -533,7 +533,12 @@
-- swallow any error
- if not continue then
+ if continue == nil then
+ -- warn if applet returns nil
+ log:warn(entry.appletName, ":free() returned nil")
+ end
+
+ if continue == false then
-- the only way for continue to be false is to have the
loaded applet have a free funtion
-- that successfully executes and returns false.
return
_______________________________________________
Jive-checkins mailing list
[email protected]
http://lists.slimdevices.com/cgi-bin/mailman/listinfo/jive-checkins
