Author: titmuss
Date: Sun Jan 27 11:16:45 2008
New Revision: 1645

URL: http://svn.slimdevices.com?rev=1645&root=Jive&view=rev
Log:
Bug: 6761
Description:
Make firmware upgrade applet (at least partly) work on the desktop to make 
debugging 
easier.
Fix "Try again" option, so it does not leave extra windows on the stack.
Fix zipfilter, so that it aborts if the stream is empty (for example an http 
404).


Modified:
    
branches/7.0/jive/src/pkg/jive_squeezeboxjive/share/applets/SetupFirmwareUpgrade/SetupFirmwareUpgradeApplet.lua
    branches/7.0/jive/src/pkg/luazipfilter/src/luazipfilter.c

Modified: 
branches/7.0/jive/src/pkg/jive_squeezeboxjive/share/applets/SetupFirmwareUpgrade/SetupFirmwareUpgradeApplet.lua
URL: 
http://svn.slimdevices.com/branches/7.0/jive/src/pkg/jive_squeezeboxjive/share/applets/SetupFirmwareUpgrade/SetupFirmwareUpgradeApplet.lua?rev=1645&root=Jive&r1=1644&r2=1645&view=diff
==============================================================================
--- 
branches/7.0/jive/src/pkg/jive_squeezeboxjive/share/applets/SetupFirmwareUpgrade/SetupFirmwareUpgradeApplet.lua
 (original)
+++ 
branches/7.0/jive/src/pkg/jive_squeezeboxjive/share/applets/SetupFirmwareUpgrade/SetupFirmwareUpgradeApplet.lua
 Sun Jan 27 11:16:45 2008
@@ -16,7 +16,6 @@
 local socket                 = require("socket")
 local lfs                    = require("lfs")
 local os                     = require("os")
-local bsp                    = require("jiveBSP")
 local coroutine               = require("coroutine")
 
 local Applet                 = require("jive.Applet")
@@ -30,7 +29,9 @@
 local Textarea               = require("jive.ui.Textarea")
 local Window                 = require("jive.ui.Window")
 local Popup                  = require("jive.ui.Popup")
+
 local Upgrade                = require("applets.SetupFirmwareUpgrade.Upgrade")
+local hasBSP, BSP            = pcall(require, "jiveBSP")
 
 local debug                  = require("jive.utils.debug")
 local log                    = 
require("jive.utils.log").logger("applets.setup")
@@ -173,7 +174,11 @@
 
 
 function _checkBattery()
-       return bsp.ioctl(23) == 0 or bsp.ioctl(17) > 830
+       if hasBSP then
+               return BSP.ioctl(23) == 0 or bsp.ioctl(17) > 830
+       else
+               return true
+       end
 end
 
 
@@ -285,7 +290,9 @@
                                                sound = "WINDOWSHOW",
                                                callback = function()
                                                                   if 
_checkBattery() then
-                                                                          
self:_upgrade():showInstead()
+                                                                          
window:hide()
+                                                                          
self:_upgrade()
+
                                                                   else
                                                                           
window:bumpRight()
                                                                   end

Modified: branches/7.0/jive/src/pkg/luazipfilter/src/luazipfilter.c
URL: 
http://svn.slimdevices.com/branches/7.0/jive/src/pkg/luazipfilter/src/luazipfilter.c?rev=1645&root=Jive&r1=1644&r2=1645&view=diff
==============================================================================
--- branches/7.0/jive/src/pkg/luazipfilter/src/luazipfilter.c (original)
+++ branches/7.0/jive/src/pkg/luazipfilter/src/luazipfilter.c Sun Jan 27 
11:16:45 2008
@@ -104,6 +104,12 @@
                case PARSE_FILE_HEADER:
                        DEBUG("PARSE FILE HEADER\n");
 
+                       if (lua_isnil(L, 1)) {
+                               /* empty file */
+                               lua_pushnil(L);
+                               return 1;
+                       }
+
                        if (end - ptr < 30 /* local file header length */) {
                                lua_pushlstring(L, (const char *)ptr, end - 
ptr);
                                lua_replace(L, lua_upvalueindex(2));

_______________________________________________
Jive-checkins mailing list
[email protected]
http://lists.slimdevices.com/cgi-bin/mailman/listinfo/jive-checkins

Reply via email to