Would you like to react to this message? Create an account in a few clicks or log in to continue.

You are not connected. Please login or register

R1832: Load memory table

2 posters

Go down  Message [Page 1 of 1]

1R1832: Load memory table Empty R1832: Load memory table Thu Aug 31, 2023 6:29 am

royqh1979



Old implementation of MemTable loadTable() is not correct:
- If memtable is backed by m_data, the table is not refreshed.
- If memtable is not backed by m_data, table refreshed by real data are not updated.

I reimplemented MemTable's loadTable() to make it efficiently work. Only cells that changed are refresh/updated.

toIntVector() return a QVector. Since QVector uses implicit data sharing, copy a QVector is not an heavy operation.   (see https://doc.qt.io/qt-5/implicit-sharing.html)
Attachments
R1832: Load memory table Attachmentmemtable.zip
You don't have permission to download attachments.
(2 Kb) Downloaded 1 times

2R1832: Load memory table Empty Re: R1832: Load memory table Thu Aug 31, 2023 9:35 am

arcachofo

arcachofo

Thank you.
Patch applied at Rev 1834.

Back to top  Message [Page 1 of 1]

Permissions in this forum:
You cannot reply to topics in this forum