About 25% of the time during an Undo is spent getting the ith EditCommand out of m_cmds: for (int i = m_cmds.count() - 1; i >= 0; --i) m_cmds[i]->unapply(); We should use an iterator.
Created attachment 6974 [details] patch
Comment on attachment 6974 [details] patch Assuming all tests pass, r=me.