----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110163/ -----------------------------------------------------------
Review request for Calligra, Oleg Kukharchuk, Adam Pigg, Smit Patel, Dimitrios Tanis, Tomasz Olszak, and Radosław Wicik. Description ------- Use KexiDB's TableViewData in the biblio table model. As a result the model is properly editable. Moreover, unlike the original solution, thanks to buffering in TableViewData all operations are no longer expensive: - row count is O(1), - data altering is O(1), - data retrieving (individual cells) is O(1) Filtering works too. The code is also relatively simple. This patch depends on https://git.reviewboard.kde.org/r/110162 (KexiTableViewColumn and KexiTableViewData from calligradb). Diffs ----- libs/kotext/BibliographyTableModel.h PRE-CREATION libs/kotext/BibliographyTableModel.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/110163/diff/ Testing ------- Features of the Bibliography Database window work properly. Not sure about memory leaks, e.g. two cases of Cursor lifetime: m_conn->deleteCursor() Thanks, Jarosław Staniek
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel