Merge pull request #583 from chrox/master

enlarge touch zone for readerfooter

.gitignore

@@ -11,6 +11,7 @@ git-rev
 tags
 test/*
 *.tar
+*.log
 spec/unit/data
 
 emu

frontend/apps/reader/modules/readerfooter.lua

@@ -27,7 +27,7 @@ local ReaderFooter = InputContainer:new{
     text_width = 0.15,
     text_font_face = "ffont",
     text_font_size = 14,
-    height = 19,
+    height = Screen:scaleByDPI(19),
 }
 
 function ReaderFooter:init()
@@ -54,11 +54,14 @@ function ReaderFooter:init()
     table.insert(horizontal_group, text_container)
     self[1] = BottomContainer:new{
         dimen = Screen:getSize(),
-        FrameContainer:new{
-            horizontal_group,
-            background = 0,
-            bordersize = 0,
-            padding = 0,
+        BottomContainer:new{
+            dimen = Geom:new{w = Screen:getWidth(), h = self.height*2},
+            FrameContainer:new{
+                horizontal_group,
+                background = 0,
+                bordersize = 0,
+                padding = 0,
+            }
         }
     }
     self.dimen = self[1]:getSize()

spec/unit/benchmark_spec.lua

@@ -0,0 +1,45 @@
+require "defaults"
+require "libs/libkoreader-luagettext"
+package.path = "?.lua;common/?.lua;frontend/?.lua"
+package.cpath = "?.so;common/?.so;/usr/lib/lua/?.so"
+
+-- global einkfb for Screen
+einkfb = require("ffi/framebuffer")
+-- do not show SDL window
+einkfb.dummy = true
+Blitbuffer = require("ffi/blitbuffer")
+util = require("ffi/util")
+
+local Screen = require("ui/screen")
+local DocSettings = require("docsettings")
+G_reader_settings = DocSettings:open(".reader")
+local DocumentRegistry = require("document/documentregistry")
+local DEBUG = require("dbg")
+
+-- screen should be inited for crengine
+Screen:init()
+
+function logDuration(filename, pageno, dur)
+    local file = io.open(filename, "a+")
+    if file then
+        if file:seek("end") == 0 then -- write the header only once
+            file:write("PAGE\tDUR\n")
+        end
+        file:write(string.format("%s\t%s\n", pageno, dur))
+        file:close()
+    end
+end
+
+describe("PDF rendering benchmark", function()
+    local sample_pdf = "spec/front/unit/data/sample.pdf"
+    local doc = DocumentRegistry:openDocument(sample_pdf)
+    for pageno = 1, doc.info.number_of_pages do
+        local secs, usecs = util.gettime()
+        assert.truthy(doc:renderPage(pageno, nil, 1, 0, 1.0, 0))
+        local nsecs, nusecs = util.gettime()
+        local dur = nsecs - secs + (nusecs - usecs) / 1000000
+        logDuration("pdf_rendering.log", pageno, dur)
+    end
+    doc:close()
+end)
+

spec/unit/document_spec.lua

@@ -50,11 +50,10 @@ describe("PDF document module", function()
 end)
 
 describe("EPUB document module", function()
-    local sample_epub = "spec/front/unit/data/leaves_of_grass.epub"
+    local sample_epub = "spec/front/unit/data/leaves.epub"
     it("should open document", function()
         doc = DocumentRegistry:openDocument(sample_epub)
         assert.truthy(doc)
-        doc:close()
     end)
     it("should close document", function()
         doc:close()