Skip to content

Commit 098fbbf

Browse files
committed
ff-qtah: Optimize deleting item
1 parent 202ee35 commit 098fbbf

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

ff-qtah/FF/Qt/TaskListWidget.hs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import Data.Map.Strict (Map, (!))
2222
import Data.Map.Strict qualified as Map
2323
import Data.Text qualified as Text
2424
import Data.Time (getCurrentTime, toGregorian, utctDay)
25-
import Foreign.Hoppy.Runtime (fromCppEnum, nullptr, toGc)
25+
import Foreign.Hoppy.Runtime (delete, fromCppEnum, nullptr, toGc)
2626
import Graphics.UI.Qtah.Core.Types qualified as Qt
2727
import Graphics.UI.Qtah.Gui.QFont (QFont)
2828
import Graphics.UI.Qtah.Gui.QFont qualified as QFont
@@ -160,10 +160,8 @@ upsertTask this keepTaskOpen entity = do
160160
deleteTaskFromUi :: TaskListWidget -> NoteId -> IO ()
161161
deleteTaskFromUi this noteId = do
162162
mExisting <- Map.lookup noteId <$> readIORef this.taskItems
163-
for_ mExisting \(oldMode, item) -> do
164-
oldModeItem <- (! oldMode) <$> readIORef this.modeItems
165-
idx <- QTreeWidgetItem.indexOfChild oldModeItem item
166-
_ <- QTreeWidgetItem.takeChild oldModeItem idx
163+
for_ mExisting \(_oldMode, item) -> do
164+
delete item
167165
modifyIORef this.taskItems $ Map.delete noteId
168166
QTreeWidget.setCurrentItem this.parent (nullptr :: QTreeWidgetItem)
169167

0 commit comments

Comments
 (0)