renun/koreader
1
0
Fork 0
mirror of https://github.com/koreader/koreader.git synced 2025-08-10 00:52:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Yeah, no, reverse iter wouldn't work.

Browse Source
This commit is contained in:
NiLuJe
2022-05-21 00:26:06 +02:00
parent ca54c7811c
commit 58777ff0d7
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
frontend/ui/uimanager.lua
View File

@@ -1167,8 +1167,8 @@ function UIManager:_checkTasks()
local task = table.remove(self._task_queue, 1)
-- ...so do it now.
-- NOTE: Said task's action might modify _task_queue.
-- To avoid race conditions, we only ever check the head of the queue (c.f., #1758).
-- FIXME: Switch to a less confusing reverse for #len loop?
-- To avoid race conditions and catch new upcoming tasks during this call,
-- we repeatedly check the head of the queue (c.f., #1758).
task.action(unpack(task.args, 1, task.argc))
else
-- As the queue is sorted in ascending order, it's safe to assume all items are currently future tasks.