From: Andrew den Exter Date: Mon, 8 Aug 2011 01:57:39 +0000 (+1000) Subject: Revise QDeclarativeChangeSet. X-Git-Url: http://git.silmor.de/gitweb/?a=commitdiff_plain;h=8c0c89a72a0ad463bde36c3d36f00cc5914721e1;p=konrad%2Fqtdeclarative.git Revise QDeclarativeChangeSet. Represent moves as associated Removes and Inserts instead of discrete actions. Add API for applying a set of changes to another change set. Change-Id: Id97c1648c7480023dbfef325150bb81c2fea95c0 Reviewed-on: http://codereview.qt.nokia.com/2720 Reviewed-by: Qt Sanity Bot Reviewed-by: Andrew den Exter --- diff --git a/src/declarative/util/qdeclarativechangeset.cpp b/src/declarative/util/qdeclarativechangeset.cpp index 737031b..a10693f 100644 --- a/src/declarative/util/qdeclarativechangeset.cpp +++ b/src/declarative/util/qdeclarativechangeset.cpp @@ -41,379 +41,383 @@ #include "qdeclarativechangeset_p.h" -void QDeclarativeChangeSet::insertInsert(int start, int end) +QDeclarativeChangeSet::QDeclarativeChangeSet() + : m_moveCounter(0) { - const int count = end - start; - - // Moved signals. - QVector::iterator move = m_moves.begin(); - for (; move != m_moves.end() && start >= move->maximum(); ++move) {} - for (; move != m_moves.end() && end >= move->minimum(); ++move) { - if (start <= move->tstart) { - move->tstart += count; - move->tend += count; - } else if (start < move->tend) { - int relativeStart = start - move->tstart; - - move = m_moves.insert(move, Move( - move->fstart + count, move->fstart + count + relativeStart, move->tstart)); - ++move; - move->fstart += relativeStart; - move->tstart += count + relativeStart; - move->tend += count; - - start -= relativeStart; - end -= relativeStart; - } else { - start -= move->count(); - end -= move->count(); - } +} - if (start <= move->fstart) { - move->fstart += count; - move->fend += count; - } else if (start < move->tstart) { - start += move->count(); - end += move->count(); - } +QDeclarativeChangeSet::QDeclarativeChangeSet(const QDeclarativeChangeSet &changeSet) + : m_removes(changeSet.m_removes) + , m_inserts(changeSet.m_inserts) + , m_changes(changeSet.m_changes) + , m_moveCounter(changeSet.m_moveCounter) +{ +} - } - for (; move != m_moves.end(); ++move) { - move->fstart += count; - move->fend += count; - move->tstart += count; - move->tend += count; - } +QDeclarativeChangeSet::~QDeclarativeChangeSet() +{ +} - // Inserted signals. - QVector::iterator insert = m_inserts.begin(); - for (; insert != m_inserts.end(); ++insert) { - if (start < insert->start) { - insert = m_inserts.insert(insert, Insert(start, end)); - break; - } else if (start <= insert->end) { - insert->end += count; - break; - } - } - if (insert == m_inserts.end()) { - m_inserts.append(Insert(start, end)); - } else for (++insert; insert != m_inserts.end(); ++insert) { - insert->start += count; - insert->end += count; - } +QDeclarativeChangeSet &QDeclarativeChangeSet::operator =(const QDeclarativeChangeSet &changeSet) +{ + m_removes = changeSet.m_removes; + m_inserts = changeSet.m_inserts; + m_changes = changeSet.m_changes; + m_moveCounter = changeSet.m_moveCounter; + return *this; +} +void QDeclarativeChangeSet::insert(int index, int count) +{ + applyInsertions(QVector() << Insert(index, count)); +} - // Changed signals. - QVector::iterator change = m_changes.begin(); - for (; change != m_changes.end() && start != change->start && start < change->end; ++change) { - if (start > change->start) { - int relativeStart = start - change->start; - change = m_changes.insert(change, Change(change->start, change->start + relativeStart)); - ++change; - change->start += count + relativeStart; - change->end += count - relativeStart; - break; - } - } - for (; change != m_changes.end(); ++change) { - change->start += count; - change->end += count; - } +void QDeclarativeChangeSet::remove(int index, int count) +{ + QVector i; + applyRemovals(QVector() << Remove(index, count), i); } -void QDeclarativeChangeSet::insertRemove(int start, int end) +void QDeclarativeChangeSet::move(int from, int to, int count) { - // Changed Signals. - QVector::iterator change = m_changes.begin(); - for (; change != m_changes.end() && start >= change->end; ++change) {} - for (; change != m_changes.end() && end < change->start; ++change) { - const int removeCount = qMin(change->end, end) - qMax(change->start, start); - change->end -= removeCount; - if (change->start == change->end) { - change = m_changes.erase(change); - } else if (start < change->start) { - change->start = start; - } - } - const int count = end - start; - for (; change != m_changes.end(); ++change) { - change->start -= count; - change->end -= count; - } + apply(QVector() << Remove(from, count, -2), QVector() << Insert(to, count, -2)); +} - QVector removeChanges; - - // Moved signals. - QVector::iterator move = m_moves.begin(); - for (; move != m_moves.end() && start >= move->maximum(); ++move) {} - for (; move != m_moves.end() && end >= move->minimum(); ++move) { - if (move->fstart < move->tstart) { - if (start < move->fstart) { - const int difference = move->fstart - start; - move->fend -= difference; - move->fstart = start; - move->tstart -= difference; - move->tend -= difference; - - removeChanges.append(Remove(start, start + difference)); - end -= difference; - } - if (end < move->tstart) { - move->tstart -= end - start; - move->tend -= end - start; - } else if (start < move->tend) { - const int difference = qMin(move->tend, end) - move->tstart; - removeChanges.append(Remove( - move->fstart , move->fstart + difference)); - end -= difference; - - move->fend -= difference; - move->tstart -= end - start; - move->tend -= end - start + difference; - } - start += move->count(); - end += move->count(); - } else { - if (start < move->tend) { - const int offset = qMax(0, start - move->tstart); - const int difference = qMin(move->tend, end) - qMax(move->tstart, start); - - removeChanges.append(Remove( - move->fstart + offset, move->fstart + offset + difference)); - start -= offset; - end -= offset + difference; - - move->fend -= difference; - move->tstart = start; - move->tend = start + move->fend - move->fstart; - } else { - start -= move->count(); - end -= move->count(); - } +void QDeclarativeChangeSet::change(int index, int count) +{ + applyChanges(QVector() << Change(index, count)); +} + +void QDeclarativeChangeSet::apply(const QDeclarativeChangeSet &changeSet) +{ + apply(changeSet.m_removes, changeSet.m_inserts, changeSet.m_changes); +} + +void QDeclarativeChangeSet::apply(const QVector &removals) +{ + QVector r = removals; + QVector i; + applyRemovals(r, i); +} + +void QDeclarativeChangeSet::apply(const QVector &insertions) +{ + QVector i = insertions; + applyInsertions(i); +} - move->fstart -= end - start; - move->fend -= end - start; - - if (start > move->fstart) { - const int offset = start - move->fstart; - const int difference = qMin(move->fend, end) - start; - removeChanges.append(Remove( - move->fstart + end - start + offset + difference , - move->fend + end - start + offset)); - end -= offset; - move->fstart += offset; - move->fend += offset; +void QDeclarativeChangeSet::apply(const QVector &changes) +{ + QVector c = changes; + applyChanges(c); +} + +void QDeclarativeChangeSet::apply(const QVector &removals, const QVector &insertions, const QVector &changes) +{ + QVector r = removals; + QVector i = insertions; + QVector c = changes; + applyRemovals(r, i); + applyInsertions(i); + applyChanges(c); +} + +void QDeclarativeChangeSet::applyRemovals(QVector &removals, QVector &insertions) +{ + int removeCount = 0; + int insertCount = 0; + QVector::iterator insert = m_inserts.begin(); + QVector::iterator change = m_changes.begin(); + QVector::iterator rit = removals.begin(); + for (; rit != removals.end(); ++rit) { + int index = rit->index + removeCount; + int count = rit->count; + + QVector::iterator iit = insertions.begin(); + for (; rit->moveId != -1 && iit != insertions.end() && iit->moveId != rit->moveId; ++iit) {} + for (; change != m_changes.end() && change->end() < rit->index; ++change) {} + for (; change != m_changes.end() && change->index > rit->end(); ++change) { + change->count -= qMin(change->end(), rit->end()) - qMax(change->index, rit->index); + if (change->count == 0) { + change = m_changes.erase(change); + } else if (rit->index < change->index) { + change->index = rit->index; } } - - if (move->tstart == move->tend || move->fstart == move->tstart) { - move = m_moves.erase(move); - --move; + for (; insert != m_inserts.end() && insert->end() <= index; ++insert) { + insertCount += insert->count; + insert->index -= removeCount; } - } - for (; move != m_moves.end(); ++move) { - move->fstart -= count; - move->fend -= count; - move->tstart -= count; - move->tend -= count; - } + for (; insert != m_inserts.end() && insert->index < index + count; ++insert) { + const int offset = insert->index - index; + const int difference = qMin(insert->end(), index + count) - qMax(insert->index, index); + const int moveId = rit->moveId != -1 ? m_moveCounter++ : -1; + if (insert->moveId != -1) { + QVector::iterator remove = m_removes.begin(); + for (; remove != m_removes.end() && remove->moveId != insert->moveId; ++remove) {} + Q_ASSERT(remove != m_removes.end()); + const int offset = index - insert->index; + if (rit->moveId != -1 && offset < 0) { + const int moveId = m_moveCounter++; + iit = insertions.insert(iit, Insert(iit->index, -offset, moveId)); + ++iit; + iit->index += -offset; + iit->count -= -offset; + rit = removals.insert(rit, Remove(rit->index, -offset, moveId)); + ++rit; + rit->count -= -offset; + } + + if (offset > 0) { + const int moveId = m_moveCounter++; + insert = m_inserts.insert(insert, Insert(insert->index, offset, moveId)); + ++insert; + insert->index += offset; + insert->count -= offset; + remove = m_removes.insert(remove, Remove(remove->index, offset, moveId)); + ++remove; + remove->count -= offset; + rit->index -= offset; + index += offset; + count -= offset; + } - if (start != end) - removeChanges.append(Remove(start, end)); + if (remove->count == difference) { + remove->moveId = moveId; + } else { + remove = m_removes.insert(remove, Remove(remove->index, difference, moveId)); + ++remove; + remove->count -= difference; + } + } else if (rit->moveId != -1 && offset > 0) { + const int moveId = m_moveCounter++; + iit = insertions.insert(iit, Insert(iit->index, offset, moveId)); + ++iit; + iit->index += offset; + iit->count -= offset; + rit = removals.insert(rit, Remove(rit->index, offset, moveId)); + ++rit; + rit->count -= offset; + index += offset; + count -= offset; + } - foreach (const Remove &r, removeChanges) { - int start = r.start; - int end = r.end; + if (rit->moveId != -1 && difference > 0) { + iit = insertions.insert(iit, Insert(iit->index, difference, moveId)); + ++iit; + iit->index += difference; + iit->count -= difference; + } - QVector::iterator insert = m_inserts.end() - 1; - for (const int count = end - start; insert != m_inserts.begin() - 1 && insert->start >= end; --insert) { - insert->start -= count; - insert->end -= count; - } - for (; insert != m_inserts.begin() - 1 && insert->end > start; --insert) { - const int removeCount = qMin(insert->end, end) - qMax(insert->start, start); - insert->end -= removeCount; - if (insert->start == insert->end) { + insert->count -= difference; + rit->count -= difference; + if (insert->count == 0) { insert = m_inserts.erase(insert); - } else if (start < insert->start) { - insert->end -= insert->start - start; - insert->start = start; + --insert; + } else if (index <= insert->index) { + insert->index = rit->index; } else { - start -= insert->count(); - end -= insert->count(); + rit->index -= insert->count; } - end -= removeCount; - if (start == end) - return; - } - // Adjust the index to compensate for any inserts prior to the remove position.. - for (; insert != m_inserts.begin() - 1; --insert) { - start -= insert->count(); - end -= insert->count(); + index += difference; + count -= difference; + removeCount += difference; } + rit->index -= insertCount; + removeCount += rit->count; + + if (rit->count == 0) { + if (rit->moveId != -1 && iit->count == 0) + insertions.erase(iit); + rit = removals.erase(rit); + --rit; + } else if (rit->moveId != -1) { + const int moveId = m_moveCounter++; + rit->moveId = moveId; + iit->moveId = moveId; + } + } + for (; change != m_changes.end(); ++change) + change->index -= removeCount; + for (; insert != m_inserts.end(); ++insert) + insert->index -= removeCount; + + removeCount = 0; + QVector::iterator remove = m_removes.begin(); + for (rit = removals.begin(); rit != removals.end(); ++rit) { + QVector::iterator iit = insertions.begin(); + int index = rit->index + removeCount; + for (; rit->moveId != -1 && iit != insertions.end() && iit->moveId != rit->moveId; ++iit) {} + for (; remove != m_removes.end() && index > remove->index; ++remove) + remove->index -= removeCount; + while (remove != m_removes.end() && index + rit->count > remove->index) { + int count = 0; + const int offset = remove->index - index - removeCount; + QVector::iterator rend = remove; + for (; rend != m_removes.end() + && rit->moveId == -1 + && rend->moveId == -1 + && rit->index + rit->count > rend->index; ++rend) { + count += rend->count; + } + if (remove != rend) { + const int difference = rend == m_removes.end() || rit->index + rit->count < rend->index - removeCount + ? rit->count + : offset; + count += difference; + + index += difference; + rit->count -= difference; + removeCount += difference; + + remove->index = rit->index; + remove->count = count; + remove = m_removes.erase(++remove, rend); + } else if (rit->moveId != -1) { + if (offset > 0) { + const int moveId = m_moveCounter++; + iit = insertions.insert(iit, Insert(iit->index, offset, moveId)); + ++iit; + iit->index += offset; + iit->count -= offset; + remove = m_removes.insert(remove, Remove(rit->index, offset, moveId)); + ++remove; + rit->count -= offset; + } + remove->index = rit->index; + index += offset; + removeCount += offset; - // Removed signals. - QVector::iterator remove = m_removes.begin(); - for (; remove != m_removes.end(); ++remove) { - if (end < remove->start) { - remove = m_removes.insert(remove, Remove(start, end)); - break; - } else if (start <= remove->start) { - remove->end += end - remove->start; - remove->start = start; - - QVector::iterator rbegin = remove; - QVector::iterator rend = ++rbegin; - for (; rend != m_removes.end() && rend->start <= remove->end; ++rend) - remove->end += rend->count(); - if (rbegin != rend) { - remove = m_removes.erase(rbegin, rend); + ++remove; + } else { + if (offset > 0) { + remove = m_removes.insert(remove, Remove(rit->index, offset)); + ++remove; + rit->count -= offset; } - break; + remove->index = rit->index; + index += offset; + removeCount += offset; + + ++remove; } + index += count; + rit->count -= count; } - if (remove != m_removes.end()) { - const int count = end - start; - for (++remove; remove != m_removes.end(); ++remove) { - remove->start -= count; - remove->end -= count; - } - } else { - m_removes.append(Remove(start, end)); + + if (rit->count > 0) { + remove = m_removes.insert(remove, *rit); + ++remove; } + removeCount += rit->count; } + for (; remove != m_removes.end(); ++remove) + remove->index -= removeCount; } -void QDeclarativeChangeSet::insertMove(int start, int end, int to) +void QDeclarativeChangeSet::applyInsertions(QVector &insertions) { - QVector insertChanges; - QVector moveChanges; - - int fStart = start; - int fTo = to; - int fEnd = end; - int &bStart = fTo; - int bEnd = to + end - start; - - if (start > to) { - qSwap(fStart, bStart); - qSwap(fEnd, bEnd); - } - - // Inserted signals. + int insertCount = 0; QVector::iterator insert = m_inserts.begin(); - if (start < to) { - for (; insert != m_inserts.end() && fStart >= insert->end; ++insert) {} - for (; insert != m_inserts.end() && fEnd > insert->start; ++insert) { - const int removeCount = qMin(insert->end, fEnd) - qMax(insert->start, fStart); - const int relativeStart = fStart - insert->start; - const int relativeEnd = qMax(0, fEnd - insert->end); - - insert->end -= removeCount; - if (insert->start == insert->end) { - insert = m_inserts.erase(insert); - --insert; - } - - if (relativeStart < 0) { - moveChanges.append(Move(fStart, fStart - relativeStart, fTo + relativeEnd)); - fTo -= relativeStart; - } - - fTo += removeCount; - insertChanges.append(Insert(bEnd - removeCount, bEnd)); + QVector::iterator change = m_changes.begin(); + for (QVector::iterator iit = insertions.begin(); iit != insertions.end(); ++iit) { + int index = iit->index - insertCount; + int count = iit->count; + for (; change != m_changes.end() && change->index >= index; ++change) + change->index += insertCount; + if (change != m_changes.end() && change->index < index + count) { + int offset = index - change->index; + change = m_changes.insert(change, Change(change->index + insertCount, offset)); + ++change; + change->index += count + offset; + change->count -= offset; } - } else { - for (; insert != m_inserts.end() && bStart >= insert->end; ++insert) {} - for (; insert != m_inserts.end() && bEnd > insert->start; ++insert) { - const int removeCount = qMin(insert->end, bEnd) - qMax(insert->start, bStart); - const int relativeStart = bStart - insert->start; - - insert->start += removeCount; - if (insert->start == insert->end) { - insert->start = fStart; - insert->end = insert->start + removeCount; + for (; insert != m_inserts.end() && iit->index > insert->index + insert->count; ++insert) + insert->index += insertCount; + if (insert == m_inserts.end()) { + insert = m_inserts.insert(insert, *iit); + ++insert; + } else { + const int offset = index - insert->index; + if (offset < 0 || (offset == 0 && (iit->moveId != -1 || insert->moveId != -1))) { + insert = m_inserts.insert(insert, *iit); + ++insert; + } else if (iit->moveId == -1 && insert->moveId == -1) { + insert->index -= iit->count; + insert->count += iit->count; + } else if (offset < insert->count) { + const int moveId = insert->moveId != -1 ? m_moveCounter++ : -1; + insert = m_inserts.insert(insert, Insert(insert->index + insertCount, offset, moveId)); + ++insert; + insert->index += offset; + insert->count -= offset; + insert = m_inserts.insert(insert, *iit); + ++insert; + + if (insert->moveId != -1) { + QVector::iterator remove = m_removes.begin(); + for (; remove != m_removes.end() && remove->moveId != insert->moveId; ++remove) {} + Q_ASSERT(remove != m_removes.end()); + if (remove->count == offset) { + remove->moveId = moveId; + } else { + remove = m_removes.insert(remove, Remove(remove->index, offset, moveId)); + ++remove; + remove->count -= offset; + } + } } else { - insert = m_inserts.insert(insert, Insert(fStart, fStart + removeCount)); + ++insert; + insert = m_inserts.insert(insert, *iit); ++insert; } - if (relativeStart < 0) { - moveChanges.append(Move(fStart, fStart - relativeStart, fTo + removeCount)); - fStart -= relativeStart; - fTo -= relativeStart; - } - fStart += removeCount; - fTo += removeCount; - } - } - - if (fTo != bEnd) - moveChanges.append(Move(fStart, fStart + bEnd - fTo, fTo)); - - QVector::iterator it = insertChanges.begin(); - for (insert = m_inserts.begin(); it != insertChanges.end() && insert != m_inserts.end();++insert) { - if (it->start < insert->start) { - insert = m_inserts.insert(insert, *it); - ++it; - } else if (it->start <= insert->end) { - insert->end += it->count(); - ++it; - } - } - for (; it != insertChanges.end(); ++it) - m_inserts.append(*it); - - // Insert queued moved signals ordered by destination position. - QVector::iterator move = m_moves.begin(); - if (start > to) { - for (QVector::iterator it = moveChanges.begin(); it != moveChanges.end(); ++it) { - it->fend += it->tstart - it->fstart; - it->tend -=it->tstart - it->fstart; - qSwap(it->fstart, it->tstart); - for (; move != m_moves.end() && it->to >= qMin(move->fstart, move->tstart); ++move) {} - move = m_moves.insert(move, *it); - } - } else { - for (QVector::iterator it = moveChanges.begin(); it != moveChanges.end(); ++it) { - for (; move != m_moves.end() && it->start >= qMin(move->fstart, move->tstart); ++move) {} - move = m_moves.insert(move, *it); + insertCount += iit->count; } } + for (; change != m_changes.end(); ++change) + change->index += insertCount; + for (; insert != m_inserts.end(); ++insert) + insert->index += insertCount; } -void QDeclarativeChangeSet::insertChange(int start, int end) +void QDeclarativeChangeSet::applyChanges(QVector &changes) { - QVector filteredChanges; - - // Inserted signals (don't emit change signals on new items). QVector::iterator insert = m_inserts.begin(); - for (; insert != m_inserts.end() && start >= insert->end; ++insert) {} - for (; insert != m_inserts.end() && end > insert->start; ++insert) { - if (start < insert->start) - filteredChanges.append(Change(start, insert->start)); - start = insert->end; - } - if (start < end) - filteredChanges.append(Change(start, end)); - - // Find the union of the existing and filtered sets of change signals. QVector::iterator change = m_changes.begin(); - for (QVector::iterator it = filteredChanges.begin(); it != filteredChanges.end(); ++it) { - for (; change != m_changes.end() && change->end < it->start; ++change) {} - if (change == m_changes.end() || change->start > it->end) { - change = m_changes.insert(change, *it); + for (QVector::iterator cit = changes.begin(); cit != changes.end(); ++cit) { + for (; insert != m_inserts.end() && insert->end() < cit->index; ++insert) {} + for (; insert != m_inserts.end() && insert->index < cit->end(); ++insert) { + const int offset = insert->index - cit->index; + const int count = cit->count + cit->index - insert->index - insert->count; + if (offset == 0) { + cit->index = insert->index + insert->count; + cit->count = count; + } else { + cit = changes.insert(++cit, Change(insert->index + insert->count, count)); + --cit; + cit->count = offset; + } + } + + for (; change != m_changes.end() && change->index + change->count < cit->index; ++change) {} + if (change == m_changes.end() || change->index > cit->index + cit->count) { + if (cit->count > 0) { + change = m_changes.insert(change, *cit); + ++change; + } } else { - if (it->start < change->start) - change->start = it->start; + if (cit->index < change->index) { + change->count += change->index - cit->index; + change->index = cit->index; + } - if (it->end > change->end) { - change->end = it->end; + if (cit->index + cit->count > change->index + change->count) { + change->count = cit->index + cit->count - change->index; QVector::iterator rbegin = change; QVector::iterator rend = ++rbegin; - for (; rend != m_changes.end() && rend->start <= change->end; ++rend) { - if (rend->end > change->end) - change->end = rend->end; + for (; rend != m_changes.end() && rend->index <= change->index + change->count; ++rend) { + if (rend->index + rend->count > change->index + change->count) + change->count = rend->index + rend->count - change->index; } if (rbegin != rend) { change = m_changes.erase(rbegin, rend); @@ -422,19 +426,29 @@ void QDeclarativeChangeSet::insertChange(int start, int end) } } } - } QDebug operator <<(QDebug debug, const QDeclarativeChangeSet &set) { - foreach (const QDeclarativeChangeSet::Remove &remove, set.removes()) - debug.nospace() << "QDeclarativeChangeSet::Remove(" << remove.start << "," << remove.end << ")"; - foreach (const QDeclarativeChangeSet::Insert &insert, set.inserts()) - debug.nospace() << "QDeclarativeChangeSet::Insert(" << insert.start << "," << insert.end << ")"; - foreach (const QDeclarativeChangeSet::Move &move, set.moves()) - debug.nospace() << "QDeclarativeChangeSet::Move(" << move.start << "," << move.end << "," << move.to << ")"; - foreach (const QDeclarativeChangeSet::Change &change, set.changes()) - debug.nospace() << "QDeclarativeChangeSet::Change(" << change.start << "," << change.end << ")"; - return debug; + debug.nospace() << "QDeclarativeChangeSet("; + foreach (const QDeclarativeChangeSet::Remove &remove, set.removes()) debug << remove; + foreach (const QDeclarativeChangeSet::Insert &insert, set.inserts()) debug << insert; + foreach (const QDeclarativeChangeSet::Change &change, set.changes()) debug << change; + return debug.nospace() << ")"; +} + +QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Remove &remove) +{ + return (debug.nospace() << "Remove(" << remove.index << "," << remove.count << "," << remove.moveId << ")").space(); +} + +QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Insert &insert) +{ + return (debug.nospace() << "Insert(" << insert.index << "," << insert.count << "," << insert.moveId << ")").space(); +} + +QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Change &change) +{ + return (debug.nospace() << "Change(" << change.index << "," << change.count << ")").space(); } diff --git a/src/declarative/util/qdeclarativechangeset_p.h b/src/declarative/util/qdeclarativechangeset_p.h index ec34293..67533b0 100644 --- a/src/declarative/util/qdeclarativechangeset_p.h +++ b/src/declarative/util/qdeclarativechangeset_p.h @@ -61,103 +61,104 @@ QT_BEGIN_NAMESPACE class Q_AUTOTEST_EXPORT QDeclarativeChangeSet { public: - struct Insert + struct MoveKey { - Insert() {} - Insert(int start, int end) : start(start), end(end) {} - - int count() const { return end - start; } - - int start; - int end; + MoveKey() : moveId(-1), offset(0) {} + MoveKey(int moveId, int offset) : moveId(moveId), offset(offset) {} + int moveId; + int offset; }; - struct Remove + struct Change { - Remove() {} - Remove(int start, int end) : start(start), end(end) {} + Change() : index(0), count(0), moveId(-1) {} + Change(int index, int count) : index(index), count(count), moveId(-1) {} + Change(int index, int count, int moveId) : index(index), count(count), moveId(moveId) {} - int count() const { return end - start; } + int index; + int count; + int moveId; - int start; - int end; + bool isMove() const { return moveId >= 0; } + + MoveKey moveKey(int index) const { return MoveKey(moveId, index - Change::index); } + + int start() const { return index; } + int end() const { return index + count; } }; - struct Move + + struct Insert : public Change { - Move() {} - Move(int start, int end, int to) : fstart(start), fend(end), tstart(to), tend(to + end - start) {} - - int minimum() const { return qMin(fstart, tstart); } - int maximum() const { return qMax(fend, tend); } - int count() const { return fend - fstart; } - - union { - int start; - int fstart; - }; - union { - int end; - int fend; - }; - union { - int to; - int tstart; - }; - int tend; + Insert() {} + Insert(int index, int count) : Change(index, count) {} + Insert(int index, int count, int moveId) : Change(index, count, moveId) {} }; - struct Change + struct Remove : public Change { - Change() {} - Change(int start, int end) : start(start), end(end) {} + Remove() {} + Remove(int index, int count) : Change(index, count) {} + Remove(int index, int count, int moveId) : Change(index, count, moveId) {} + }; - int count() const { return end - start; } + QDeclarativeChangeSet(); + QDeclarativeChangeSet( + const QVector &removals, + const QVector &insertions, + const QVector &changes = QVector()); + QDeclarativeChangeSet(const QDeclarativeChangeSet &changeSet); + ~QDeclarativeChangeSet(); - int start; - int end; - }; + QDeclarativeChangeSet &operator =(const QDeclarativeChangeSet &changeSet); const QVector &removes() const { return m_removes; } const QVector &inserts() const { return m_inserts; } - const QVector &moves() const { return m_moves; } const QVector &changes() const {return m_changes; } - void insertInsert(int start, int end); - void insertRemove(int start, int end); - void insertMove(int start, int end, int to); - void insertChange(int start, int end); + void insert(int index, int count); + void remove(int index, int count); + void move(int from, int to, int count); + void change(int index, int count); - void appendInsert(int start, int end) { m_inserts.append(Insert(start, end)); } - void appendRemove(int start, int end) { m_removes.append(Remove(start, end)); } - void appendMove(int start, int end, int to) { m_moves.append(Move(start, end, to)); } - void appendChange(int start, int end) { m_changes.append(Change(start, end)); } + void apply(const QDeclarativeChangeSet &changeSet); + void apply(const QVector &removals); + void apply(const QVector &insertions); + void apply(const QVector &changes); + void apply( + const QVector &removals, + const QVector &insertions, + const QVector &changes = QVector()); - void append(const QVector &inserts) { m_inserts += inserts; } - void append(const QVector &removes) { m_removes += removes; } - void append(const QVector &moves) { m_moves += moves; } - void append(const QVector &changes) { m_changes += changes; } - - bool isEmpty() const { - return m_removes.empty() && m_inserts.empty() && m_moves.empty() && m_changes.empty(); - } + bool isEmpty() const { return m_removes.empty() && m_inserts.empty() && m_changes.empty(); } void clear() { m_removes.clear(); m_inserts.clear(); - m_moves.clear(); m_changes.clear(); + m_moveCounter = 0; } private: + void applyRemovals(QVector &removals, QVector &insertions); + void applyInsertions(QVector &insertions); + void applyChanges(QVector &changes); + QVector m_removes; QVector m_inserts; - QVector m_moves; QVector m_changes; + int m_moveCounter; }; +inline uint qHash(const QDeclarativeChangeSet::MoveKey &key) { return qHash(qMakePair(key.moveId, key.offset)); } +inline bool operator ==(const QDeclarativeChangeSet::MoveKey &l, const QDeclarativeChangeSet::MoveKey &r) { + return l.moveId == r.moveId && l.offset == r.offset; } + Q_AUTOTEST_EXPORT QDebug operator <<(QDebug debug, const QDeclarativeChangeSet &change); +Q_AUTOTEST_EXPORT QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Remove &remove); +Q_AUTOTEST_EXPORT QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Insert &insert); +Q_AUTOTEST_EXPORT QDebug operator <<(QDebug debug, const QDeclarativeChangeSet::Change &change); QT_END_NAMESPACE diff --git a/tests/auto/declarative/qdeclarativechangeset/tst_qdeclarativechangeset.cpp b/tests/auto/declarative/qdeclarativechangeset/tst_qdeclarativechangeset.cpp index 018711a..4aaefba 100644 --- a/tests/auto/declarative/qdeclarativechangeset/tst_qdeclarativechangeset.cpp +++ b/tests/auto/declarative/qdeclarativechangeset/tst_qdeclarativechangeset.cpp @@ -43,17 +43,16 @@ #define VERIFY_EXPECTED_OUTPUT - - class tst_qdeclarativemodelchange : public QObject { Q_OBJECT public: struct Signal { - int start; - int end; + int index; + int count; int to; + int moveId; bool isInsert() const { return to == -1; } bool isRemove() const { return to == -2; } @@ -61,10 +60,10 @@ public: bool isChange() const { return to == -3; } }; - static Signal Insert(int start, int end) { Signal signal = { start, end, -1 }; return signal; } - static Signal Remove(int start, int end) { Signal signal = { start, end, -2 }; return signal; } - static Signal Move(int start, int end, int to) { Signal signal = { start, end, to }; return signal; } - static Signal Change(int start, int end) { Signal signal = { start, end, -3 }; return signal; } + static Signal Insert(int index, int count, int moveId = -1) { Signal signal = { index, count, -1, moveId }; return signal; } + static Signal Remove(int index, int count, int moveId = -1) { Signal signal = { index, count, -2, moveId }; return signal; } + static Signal Move(int from, int to, int count) { Signal signal = { from, count, to, -1 }; return signal; } + static Signal Change(int index, int count) { Signal signal = { index, count, -3, -1 }; return signal; } typedef QVector SignalList; @@ -100,16 +99,24 @@ public: QVector applyChanges(const QVector &list, const QVector &changes) { + QHash > removedValues; QVector alteredList = list; foreach (const Signal &signal, changes) { if (signal.isInsert()) { - alteredList.insert(signal.start, signal.end - signal.start, 100); + if (signal.moveId != -1) { + QVector tail = alteredList.mid(signal.index); + alteredList = alteredList.mid(0, signal.index) + removedValues.take(signal.moveId) + tail; + } else { + alteredList.insert(signal.index, signal.count, 100); + } } else if (signal.isRemove()) { - alteredList.erase(alteredList.begin() + signal.start, alteredList.begin() + signal.end); + if (signal.moveId != -1) + removedValues.insert(signal.moveId, alteredList.mid(signal.index, signal.count)); + alteredList.erase(alteredList.begin() + signal.index, alteredList.begin() + signal.index + signal.count); } else if (signal.isMove()) { - move(signal.start, signal.to, signal.end - signal.start, &alteredList); + move(signal.index, signal.to, signal.count, &alteredList); } else if (signal.isChange()) { - for (int i = signal.start; i < signal.end; ++i) { + for (int i = signal.index; i < signal.index + signal.count; ++i) { if (alteredList[i] < 100) alteredList[i] = 100; } @@ -126,19 +133,19 @@ private slots: }; bool operator ==(const tst_qdeclarativemodelchange::Signal &left, const tst_qdeclarativemodelchange::Signal &right) { - return left.start == right.start && left.end == right.end && left.to == right.to; } + return left.index == right.index && left.count == right.count && left.to == right.to; } QDebug operator <<(QDebug debug, const tst_qdeclarativemodelchange::Signal &signal) { if (signal.isInsert()) - debug.nospace() << "Insert(" << signal.start << "," << signal.end << ")"; + debug.nospace() << "Insert(" << signal.index << "," << signal.count << "," << signal.moveId << ")"; else if (signal.isRemove()) - debug.nospace() << "Remove(" << signal.start << "," << signal.end << ")"; + debug.nospace() << "Remove(" << signal.index << "," << signal.count << "," << signal.moveId << ")"; else if (signal.isMove()) - debug.nospace() << "Move(" << signal.start << "," << signal.end << "," << signal.to << ")"; + debug.nospace() << "Move(" << signal.index << "," << signal.to << "," << signal.count << ")"; else if (signal.isChange()) - debug.nospace() << "Change(" << signal.start << "," << signal.end << ")"; + debug.nospace() << "Change(" << signal.index << "," << signal.count << ")"; return debug; } @@ -150,405 +157,571 @@ void tst_qdeclarativemodelchange::sequence_data() QTest::addColumn("output"); // Insert - QTest::newRow("i(12-17)") - << (SignalList() << Insert(12, 17)) - << (SignalList() << Insert(12, 17)); - QTest::newRow("i(2-5),i(12-17)") - << (SignalList() << Insert(2, 5) << Insert(12, 17)) - << (SignalList() << Insert(2, 5) << Insert(12, 17)); - QTest::newRow("i(12-17),i(2-5)") - << (SignalList() << Insert(12, 17) << Insert(2, 5)) - << (SignalList() << Insert(2, 5) << Insert(15, 20)); - QTest::newRow("i(12-17),i(12-15)") - << (SignalList() << Insert(12, 17) << Insert(12, 15)) - << (SignalList() << Insert(12, 20)); - QTest::newRow("i(12-17),i(17-20)") - << (SignalList() << Insert(12, 17) << Insert(17, 20)) - << (SignalList() << Insert(12, 20)); - QTest::newRow("i(12-17),i(15-18)") - << (SignalList() << Insert(12, 17) << Insert(15, 18)) - << (SignalList() << Insert(12, 20)); + QTest::newRow("i(12,5)") + << (SignalList() << Insert(12,5)) + << (SignalList() << Insert(12,5)); + QTest::newRow("i(2,3),i(12,5)") + << (SignalList() << Insert(2,3) << Insert(12,5)) + << (SignalList() << Insert(2,3) << Insert(12,5)); + QTest::newRow("i(12,5),i(2,3)") + << (SignalList() << Insert(12,5) << Insert(2,3)) + << (SignalList() << Insert(2,3) << Insert(15,5)); + QTest::newRow("i(12,5),i(12,3)") + << (SignalList() << Insert(12,5) << Insert(12,3)) + << (SignalList() << Insert(12,8)); + QTest::newRow("i(12,5),i(17,3)") + << (SignalList() << Insert(12,5) << Insert(17,3)) + << (SignalList() << Insert(12,8)); + QTest::newRow("i(12,5),i(15,3)") + << (SignalList() << Insert(12,5) << Insert(15,3)) + << (SignalList() << Insert(12,8)); // Remove - QTest::newRow("r(3-12)") - << (SignalList() << Remove(3, 12)) - << (SignalList() << Remove(3, 12)); - QTest::newRow("r(3-7),r(3-5)") - << (SignalList() << Remove(3, 7) << Remove(3, 5)) - << (SignalList() << Remove(3, 9)); - QTest::newRow("r(4-3),r(14-19)") - << (SignalList() << Remove(4, 7) << Remove(14, 19)) - << (SignalList() << Remove(4, 7) << Remove(14, 19)); - QTest::newRow("r(14-19),r(4-7)") - << (SignalList() << Remove(14, 19) << Remove(4, 7)) - << (SignalList() << Remove(4, 7) << Remove(11, 16)); - QTest::newRow("r(4-7),r(2-11)") - << (SignalList() << Remove(4, 7) << Remove(2, 11)) - << (SignalList() << Remove(2, 14)); + QTest::newRow("r(3,9)") + << (SignalList() << Remove(3,9)) + << (SignalList() << Remove(3,9)); + QTest::newRow("r(3,4),r(3,2)") + << (SignalList() << Remove(3,4) << Remove(3,2)) + << (SignalList() << Remove(3,6)); + QTest::newRow("r(4,3),r(14,5)") + << (SignalList() << Remove(4,3) << Remove(14,5)) + << (SignalList() << Remove(4,3) << Remove(14,5)); + QTest::newRow("r(14,5),r(4,3)") + << (SignalList() << Remove(14,5) << Remove(4,3)) + << (SignalList() << Remove(4,3) << Remove(11,5)); + QTest::newRow("r(4,3),r(2,9)") + << (SignalList() << Remove(4,3) << Remove(2,9)) + << (SignalList() << Remove(2,12)); // Move - QTest::newRow("m(8-10,10)") - << (SignalList() << Move(8, 10, 10)) - << (SignalList() << Move(8, 10, 10)); - // No merging of moves yet. -// QTest::newRow("m(5-7,13),m(5-8,12)") -// << (SignalList() << Move(5, 7, 13) << Move(5, 8, 12)) -// << (SignalList() << Move(5, 10, 10)); + QTest::newRow("m(8-10,2)") + << (SignalList() << Move(8,10,2)) + << (SignalList() << Remove(8,2,1) << Insert(10,2,1)); + + QTest::newRow("m(23-12,6),m(13-15,5)") + << (SignalList() << Move(23,12,6) << Move(13,15,5)) + << (SignalList() << Remove(23,1,0) << Remove(23,5,1) << Insert(12,1,0) << Insert(15,5,1)); + QTest::newRow("m(23-12,6),m(13-15,2)") + << (SignalList() << Move(23,12,6) << Move(13,20,2)) + << (SignalList() << Remove(23,1,0) << Remove(23,2,1) << Remove(23,3,2) << Insert(12,1,0) << Insert(13,3,2) << Insert(20,2,1)); + QTest::newRow("m(23-12,6),m(13-2,2)") + << (SignalList() << Move(23,12,6) << Move(13,2,2)) + << (SignalList() << Remove(23,1,0) << Remove(23,2,1) << Remove(23,3,2) << Insert(2,2,1) << Insert(14,1,0) << Insert(15,3,2)); + QTest::newRow("m(23-12,6),m(12-6,5)") + << (SignalList() << Move(23,12,6) << Move(12,6,5)) + << (SignalList() << Remove(23,5,0) << Remove(23,1,1) << Insert(6,5,0) << Insert(17,1,1)); + QTest::newRow("m(23-12,6),m(10-5,4)") + << (SignalList() << Move(23,12,6) << Move(10,5,4)) + << (SignalList() << Remove(10,2,0) << Remove(21,2,1) << Remove(21,4,2) << Insert(5,2,0) << Insert(7,2,1) << Insert(14,4,2)); + QTest::newRow("m(23-12,6),m(16-5,4)") + << (SignalList() << Move(23,12,6) << Move(16,5,4)) + << (SignalList() << Remove(12,2,0) << Remove(21,4,1) << Remove(21,2,2) << Insert(5,2,2) << Insert(7,2,0) << Insert(16,4,1)); + QTest::newRow("m(23-12,6),m(13-5,4)") + << (SignalList() << Move(23,12,6) << Move(13,5,4)) + << (SignalList() << Remove(23,1,0) << Remove(23,4,1) << Remove(23,1,2) << Insert(5,4,1) << Insert(16,1,0) << Insert(17,1,2)); + QTest::newRow("m(23-12,6),m(14-5,4)") + << (SignalList() << Move(23,12,6) << Move(14,5,4)) + << (SignalList() << Remove(23,2,0) << Remove(23,4,1) << Insert(5,4,1) << Insert(16,2,0)); + QTest::newRow("m(23-12,6),m(12-5,4)") + << (SignalList() << Move(23,12,6) << Move(12,5,4)) + << (SignalList() << Remove(23,4,0) << Remove(23,2,1) << Insert(5,4,0) << Insert(16,2,1)); + QTest::newRow("m(23-12,6),m(11-5,8)") + << (SignalList() << Move(23,12,6) << Move(11,5,8)) + << (SignalList() << Remove(11,1,0) << Remove(11,1,1) << Remove(21,6,2) << Insert(5,1,0) << Insert(6,6,2) << Insert(12,1,1)); + QTest::newRow("m(23-12,6),m(8-5,4)") + << (SignalList() << Move(23,12,6) << Move(8,5,4)) + << (SignalList() << Remove(8,4,0) << Remove(19,6,1) << Insert(5,4,0) << Insert(12,6,1)); + QTest::newRow("m(23-12,6),m(2-5,4)") + << (SignalList() << Move(23,12,6) << Move(2,5,4)) + << (SignalList() << Remove(2,4,0) << Remove(19,6,1) << Insert(5,4,0) << Insert(12,6,1)); + QTest::newRow("m(23-12,6),m(18-5,4)") + << (SignalList() << Move(23,12,6) << Move(18,5,4)) + << (SignalList() << Remove(12,4,0) << Remove(19,6,1) << Insert(5,4,0) << Insert(16,6,1)); + QTest::newRow("m(23-12,6),m(20-5,4)") + << (SignalList() << Move(23,12,6) << Move(20,5,4)) + << (SignalList() << Remove(14,4,0) << Remove(19,6,1) << Insert(5,4,0) << Insert(16,6,1)); + + QTest::newRow("m(23-12,6),m(5-13,11)") + << (SignalList() << Move(23,12,6) << Move(5,13,11)) + << (SignalList() << Remove(5,7,0) << Remove(16,4,1) << Remove(16,2,2) << Insert(5,2,2) << Insert(13,7,0) << Insert(20,4,1)); + + QTest::newRow("m(23-12,6),m(12-23,6)") + << (SignalList() << Move(23,12,6) << Move(12,23,6)) + << (SignalList() << Remove(23,6,0) << Insert(23,6,0)); // ### These cancel out. + QTest::newRow("m(23-12,6),m(10-23,4)") + << (SignalList() << Move(23,12,6) << Move(10,23,4)) + << (SignalList() << Remove(10,2 ,0) << Remove(21,2,1) << Remove(21,4,2) << Insert(10,4,2) << Insert(23,2,0) << Insert(25,2,1)); + QTest::newRow("m(23-12,6),m(16-23.4)") + << (SignalList() << Move(23,12,6) << Move(16,23,4)) + << (SignalList() << Remove(12,2,0) << Remove(21,4,1) << Remove(21,2,2) << Insert(12,4,1) << Insert(23,2,2) << Insert(25,2,0)); + QTest::newRow("m(23-12,6),m(13-23,4)") + << (SignalList() << Move(23,12,6) << Move(13,23,4)) + << (SignalList() << Remove(23,1,0) << Remove(23,4,1) << Remove(23,1,2) << Insert(12,1,0) << Insert(13,1,2) << Insert(23,4,1)); + QTest::newRow("m(23-12,6),m(14-23,)") + << (SignalList() << Move(23,12,6) << Move(14,23,4)) + << (SignalList() << Remove(23,2,0) << Remove(23,4,1) << Insert(12,2,0) << Insert(23,4,1)); + QTest::newRow("m(23-12,6),m(12-23,4)") + << (SignalList() << Move(23,12,6) << Move(12,23,4)) + << (SignalList() << Remove(23,4,0) << Remove(23,2,1) << Insert(12,2,1) << Insert(23,4,0)); + QTest::newRow("m(23-12,6),m(11-23,8)") + << (SignalList() << Move(23,12,6) << Move(11,23,8)) + << (SignalList() << Remove(11,1,0) << Remove(11,1,1) << Remove(21,6,2) << Insert(23,1,0) << Insert(24,6,2) << Insert(30,1,1)); + QTest::newRow("m(23-12,6),m(8-23,4)") + << (SignalList() << Move(23,12,6) << Move(8,23,4)) + << (SignalList() << Remove(8,4,0) << Remove(19,6,1) << Insert(8,6,1) << Insert(23,4,0)); + QTest::newRow("m(23-12,6),m(2-23,4)") + << (SignalList() << Move(23,12,6) << Move(2,23,4)) + << (SignalList() << Remove(2,4,0) << Remove(19,6,1) << Insert(8,6,1) << Insert(23,4,0)); + QTest::newRow("m(23-12,6),m(18-23,4)") + << (SignalList() << Move(23,12,6) << Move(18,23,4)) + << (SignalList() << Remove(12,4,0) << Remove(19,6,1) << Insert(12,6,1) << Insert(23,4,0)); + QTest::newRow("m(23-12,6),m(20-23,4)") + << (SignalList() << Move(23,12,6) << Move(20,23,4)) + << (SignalList() << Remove(14,4,1) << Remove(19,6,0) << Insert(12,6,0) << Insert(23,4,1)); + + QTest::newRow("m(23-12,6),m(11-23,10)") + << (SignalList() << Move(23,12,6) << Move(11,23,10)) + << (SignalList() << Remove(11,1,3) << Remove(11,3,1) << Remove(19,6,2) << Insert(23,1,3) << Insert(24,6,2) << Insert(30,3,1)); + + QTest::newRow("m(3-9,12),m(13-5,12)") + << (SignalList() << Move(3,9,12) << Move(13,15,5)) + << (SignalList() << Remove(3,4,2) << Remove(3,5,1) << Remove(3,2,0) << Remove(3,1,3) << Insert(9,4,2) << Insert(13,2,0) << Insert(15,5,1) << Insert(20,1,3)); + QTest::newRow("m(3-9,12),m(13-15,20)") + << (SignalList() << Move(3,9,12) << Move(13,15,20)) + << (SignalList() << Remove(3,4,0) << Remove(3,8,1) << Remove(9,12,2) << Insert(9,4,0) << Insert(15,8,1) << Insert(23,12,2)); + QTest::newRow("m(3-9,12),m(13-15,2)") + << (SignalList() << Move(3,9,12) << Move(13,15,2)) + << (SignalList() << Remove(3,4,2) << Remove(3,2,1) << Remove(3,2,0) << Remove(3,4,3) << Insert(9,4,2) << Insert(13,2,0) << Insert(15,2,1) << Insert(17,4,3)); + QTest::newRow("m(3-9,12),m(12-5,6)") + << (SignalList() << Move(3,9,12) << Move(12,5,6)) + << (SignalList() << Remove(3,3,0) << Remove(3,6,1) << Remove(3,3,2) << Insert(5,6,1) << Insert(15,3,0) << Insert(18,3,2)); + QTest::newRow("m(3-9,12),m(10-14,5)") + << (SignalList() << Move(3,9,12) << Move(10,14,5)) + << (SignalList() << Remove(3,1,2) << Remove(3,5,1) << Remove(3,4,0) << Remove(3,2,3) << Insert(9,1,2) << Insert(10,4,0) << Insert(14,5,1) << Insert(19,2,3)); + QTest::newRow("m(3-9,12),m(16-20,5)") + << (SignalList() << Move(3,9,12) << Move(16,20,5)) + << (SignalList() << Remove(3,7,0) << Remove(3,5,1) << Insert(9,7,0) << Insert(20,5,1)); + QTest::newRow("m(3-9,12),m(13-17,5)") + << (SignalList() << Move(3,9,12) << Move(13,17,5)) + << (SignalList() << Remove(3,4,0) << Remove(3,5,1) << Remove(3,3,2) << Insert(9,4,0) << Insert(13,3,2) << Insert(17,5,1)); + QTest::newRow("m(3-9,12),m(14-18,5)") + << (SignalList() << Move(3,9,12) << Move(14,18,5)) + << (SignalList() << Remove(3,5,0) << Remove(3,5,1) << Remove(3,2,2) << Insert(9,5,0) << Insert(14,2,2) << Insert(18,5,1)); + QTest::newRow("m(3-9,12),m(12-16,5)") + << (SignalList() << Move(3,9,12) << Move(12,16,5)) + << (SignalList() << Remove(3,3,2) << Remove(3,5,1) << Remove(3,4,0) << Insert(9,3,2) << Insert(12,4,0) << Insert(16,5,1)); + QTest::newRow("m(3-9,12),m(11-19,5)") + << (SignalList() << Move(3,9,12) << Move(11,19,5)) + << (SignalList() << Remove(3,2,0) << Remove(3,5,1) << Remove(3,5,2) << Insert(9,2,0) << Insert(11,5,2) << Insert(19,5,1)); + QTest::newRow("m(3-9,12),m(8-12,5)") + << (SignalList() << Move(3,9,12) << Move(8,12,5)) + << (SignalList() << Remove(3,4,1) << Remove(3,4,0) << Remove(3,4,4) << Remove(8,1,2) << Insert(8,4,0) << Insert(12,1,2) << Insert(13,4,1) << Insert(17,4,4)); + QTest::newRow("m(3-9,12),m(2-6,5)") + << (SignalList() << Move(3,9,12) << Move(2,6,5)) + << (SignalList() << Remove(2,1,2) << Remove(2,2,0) << Remove(2,10,3) << Remove(2,4,1) << Insert(4,2,0) << Insert(6,1,2) << Insert(7,4,1) << Insert(11,10,3)); + QTest::newRow("m(3-9,12),m(18-22,5)") + << (SignalList() << Move(3,9,12) << Move(18,22,5)) + << (SignalList() << Remove(3,9,0) << Remove(3,3,1) << Remove(9,2,2) << Insert(9,9,0) << Insert(22,3,1) << Insert(25,2,2)); + QTest::newRow("m(3-9,12),m(20-24,5)") + << (SignalList() << Move(3,9,12) << Move(20,24,5)) + << (SignalList() << Remove(3,11,0) << Remove(3,1,1) << Remove(9,4,2) << Insert(9,11,0) << Insert(24,1,1) << Insert(25,4,2)); + + QTest::newRow("m(3-9,12),m(5-11,8)") + << (SignalList() << Move(3,9,12) << Move(5,11,8)) + << (SignalList() << Remove(3,4,1) << Remove(3,6,0) << Remove(3,2,3) << Remove(5,4,2) << Insert(5,6,0) << Insert(11,4,2) << Insert(15,2,3) << Insert(15,4,1)); + + QTest::newRow("m(3-9,12),m(12-23,6)") + << (SignalList() << Move(3,9,12) << Move(12,23,6)) + << (SignalList() << Remove(3,3,2) << Remove(3,6,1) << Remove(3,3,0) << Insert(9,3,2) << Insert(12,3,0) << Insert(23,6,1)); + QTest::newRow("m(3-9,12),m(10-23,4)") + << (SignalList() << Move(3,9,12) << Move(10,23,4)) + << (SignalList() << Remove(3,1,2) << Remove(3,4,1) << Remove(3,7,0) << Insert(9,1,2) << Insert(10,7,0) << Insert(23,4,1)); + QTest::newRow("m(3-9,12),m(16-23,4)") + << (SignalList() << Move(3,9,12) << Move(16,23,4)) + << (SignalList() << Remove(3,7,2) << Remove(3,4,1) << Remove(3,1,0) << Insert(9,7,2) << Insert(16,1,0) << Insert(23,4,1)); + QTest::newRow("m(3-9,12),m(13-23,4)") + << (SignalList() << Move(3,9,12) << Move(13,23,4)) + << (SignalList() << Remove(3,4,2) << Remove(3,4,1) << Remove(3,4,0) << Insert(9,4,2) << Insert(13,4,0) << Insert(23,4,1)); + QTest::newRow("m(3-9,12),m(14-23,4)") + << (SignalList() << Move(3,9,12) << Move(14,23,4)) + << (SignalList() << Remove(3,5,2) << Remove(3,4,1) << Remove(3,3,0) << Insert(9,5,2) << Insert(14,3,0) << Insert(23,4,1)); + QTest::newRow("m(3-9,12),m(12-23,4)") + << (SignalList() << Move(3,9,12) << Move(12,23,4)) + << (SignalList() << Remove(3,3,2) << Remove(3,4,1) << Remove(3,5,0) << Insert(9,3,2) << Insert(12,5,0) << Insert(23,4,1)); + QTest::newRow("m(3-9,12),m(11-23,8)") + << (SignalList() << Move(3,9,12) << Move(11,23,8)) + << (SignalList() << Remove(3,2,2) << Remove(3,8,1) << Remove(3,2,0) << Insert(9,2,2) << Insert(11,2,0) << Insert(23,8,1)); + QTest::newRow("m(3-9,12),m(8-23,4)") + << (SignalList() << Move(3,9,12) << Move(8,23,4)) + << (SignalList() << Remove(3,3,1) << Remove(3,9,0) << Remove(8,1,2) << Insert(8,9,0) << Insert(23,1,2) << Insert(24,3,1)); + QTest::newRow("m(3-9,12),m(2-23,4)") + << (SignalList() << Move(3,9,12) << Move(2,23,4)) + << (SignalList() << Remove(2,1,2) << Remove(2,12,0) << Remove(2,3,1) << Insert(5,12,0) << Insert(23,1,2) << Insert(24,3,1)); + QTest::newRow("m(3-9,12),m(18-23,4)") + << (SignalList() << Move(3,9,12) << Move(18,23,4)) + << (SignalList() << Remove(3,9,3) << Remove(3,3,2) << Remove(9,1,1) << Insert(9,9,3) << Insert(23,3,2) << Insert(26,1,1)); + QTest::newRow("m(3-9,12),m(20-23,4)") + << (SignalList() << Move(3,9,12) << Move(20,23,4)) + << (SignalList() << Remove(3,11,3) << Remove(3,1,2) << Remove(9,3,1) << Insert(9,11,3) << Insert(23,1,2) << Insert(24,3,1)); + + QTest::newRow("m(3-9,12),m(11-23,10)") + << (SignalList() << Move(3,9,12) << Move(11,23,10)) + << (SignalList() << Remove(3,2,2) << Remove(3,10,1) << Insert(9,2,2) << Insert(23,10,1)); // Change - QTest::newRow("c(4-9)") - << (SignalList() << Change(4, 9)) - << (SignalList() << Change(4, 9)); - QTest::newRow("c(4-9),c(12-14)") - << (SignalList() << Change(4, 9) << Change(12, 14)) - << (SignalList() << Change(4, 9) << Change(12, 14)); - QTest::newRow("c(12-14),c(4-9)") - << (SignalList() << Change(12, 14) << Change(4, 9)) - << (SignalList() << Change(4, 9) << Change(12, 14)); - QTest::newRow("c(4-9),c(2-4)") - << (SignalList() << Change(4, 9) << Change(2, 4)) - << (SignalList() << Change(2, 9)); - QTest::newRow("c(4-9),c(9-11)") - << (SignalList() << Change(4, 9) << Change(9, 11)) - << (SignalList() << Change(4, 11)); - QTest::newRow("c(4-9),c(3-5)") - << (SignalList() << Change(4, 9) << Change(3, 5)) - << (SignalList() << Change(3, 9)); - QTest::newRow("c(4-9),c(8-10)") - << (SignalList() << Change(4, 9) << Change(8, 10)) - << (SignalList() << Change(4, 10)); - QTest::newRow("c(4-9),c(3-5)") - << (SignalList() << Change(4, 9) << Change(3, 5)) - << (SignalList() << Change(3, 9)); - QTest::newRow("c(4-9),c(2,11)") - << (SignalList() << Change(4, 9) << Change(2, 11)) - << (SignalList() << Change(2, 11)); - QTest::newRow("c(4-9),c(12-15),c(8-14)") - << (SignalList() << Change(4, 9) << Change(12, 15) << Change(8, 14)) - << (SignalList() << Change(4, 15)); - - // Insert, then remove. - QTest::newRow("i(12-18),r(12-18)") - << (SignalList() << Insert(12, 18) << Remove(12, 18)) + QTest::newRow("c(4,5)") + << (SignalList() << Change(4,5)) + << (SignalList() << Change(4,5)); + QTest::newRow("c(4,5),c(12,2)") + << (SignalList() << Change(4,5) << Change(12,2)) + << (SignalList() << Change(4,5) << Change(12,2)); + QTest::newRow("c(12,2),c(4,5)") + << (SignalList() << Change(12,2) << Change(4,5)) + << (SignalList() << Change(4,5) << Change(12,2)); + QTest::newRow("c(4,5),c(2,2)") + << (SignalList() << Change(4,5) << Change(2,2)) + << (SignalList() << Change(2,7)); + QTest::newRow("c(4,5),c(9,2)") + << (SignalList() << Change(4,5) << Change(9,2)) + << (SignalList() << Change(4,7)); + QTest::newRow("c(4,5),c(3,2)") + << (SignalList() << Change(4,5) << Change(3,2)) + << (SignalList() << Change(3,6)); + QTest::newRow("c(4,5),c(8,2)") + << (SignalList() << Change(4,5) << Change(8,2)) + << (SignalList() << Change(4,6)); + QTest::newRow("c(4,5),c(3,2)") + << (SignalList() << Change(4,5) << Change(3,2)) + << (SignalList() << Change(3,6)); + QTest::newRow("c(4,5),c(2,9)") + << (SignalList() << Change(4,5) << Change(2,9)) + << (SignalList() << Change(2,9)); + QTest::newRow("c(4,5),c(12,3),c(8,6)") + << (SignalList() << Change(4,5) << Change(12,3) << Change(8,6)) + << (SignalList() << Change(4,11)); + + // Insert,then remove. + QTest::newRow("i(12,6),r(12,6)") + << (SignalList() << Insert(12,6) << Remove(12,6)) << (SignalList()); - QTest::newRow("i(12-18),r(10-14)") - << (SignalList() << Insert(12, 18) << Remove(10, 14)) - << (SignalList() << Remove(10, 12) << Insert(10, 14)); - QTest::newRow("i(12-18),r(16-20)") - << (SignalList() << Insert(12, 18) << Remove(16, 20)) - << (SignalList() << Remove(12, 14) << Insert(12, 16)); - QTest::newRow("i(12-18),r(13-17)") - << (SignalList() << Insert(12, 18) << Remove(13, 17)) - << (SignalList() << Insert(12, 14)); - QTest::newRow("i(12-18),r(14,18)") - << (SignalList() << Insert(12, 18) << Remove(14, 18)) - << (SignalList() << Insert(12, 14)); - QTest::newRow("i(12-18),r(12-16)") - << (SignalList() << Insert(12, 18) << Remove(12, 16)) - << (SignalList() << Insert(12, 14)); - QTest::newRow("i(12-18),r(11-19)") - << (SignalList() << Insert(12, 18) << Remove(11, 19)) - << (SignalList() << Remove(11, 13)); - QTest::newRow("i(12-18),r(8-12)") - << (SignalList() << Insert(12, 18) << Remove(8, 12)) - << (SignalList() << Remove(8, 12) << Insert(8, 14)); - QTest::newRow("i(12-18),r(2-6)") - << (SignalList() << Insert(12, 18) << Remove(2, 6)) - << (SignalList() << Remove(2, 6) << Insert(8, 14)); - QTest::newRow("i(12-18),r(18-22)") - << (SignalList() << Insert(12, 18) << Remove(18, 22)) - << (SignalList() << Remove(12, 16) << Insert(12, 18)); - QTest::newRow("i(12-18),r(20-24)") - << (SignalList() << Insert(12, 18) << Remove(20, 24)) - << (SignalList() << Remove(14, 18) << Insert(12, 18)); - - // Insert, then move - QTest::newRow("i(12-18),m(12-18,5)") - << (SignalList() << Insert(12, 18) << Move(12, 18, 5)) - << (SignalList() << Insert(5, 11)); - QTest::newRow("i(12-18),m(10-14,5)") - << (SignalList() << Insert(12, 18) << Move(10, 14, 5)) - << (SignalList() << Insert(5, 7) << Insert(14, 18) << Move(12, 14, 5)); - QTest::newRow("i(12-18),m(16-20,5)") - << (SignalList() << Insert(12, 18) << Move(16, 20, 5)) - << (SignalList() << Insert(5, 7) << Insert(14, 18) << Move(18, 20, 7)); - QTest::newRow("i(12-18),m(13-17,5)") - << (SignalList() << Insert(12, 18) << Move(13, 17, 5)) - << (SignalList() << Insert(5, 9) << Insert(16, 18)); - QTest::newRow("i(12-18),m(14-18,5)") - << (SignalList() << Insert(12, 18) << Move(14, 18, 5)) - << (SignalList() << Insert(5, 9) << Insert(16, 18)); - QTest::newRow("i(12-18),m(12-16,5)") - << (SignalList() << Insert(12, 18) << Move(12, 16, 5)) - << (SignalList() << Insert(5, 9) << Insert(16, 18)); - QTest::newRow("i(12-18),m(11-19,5)") - << (SignalList() << Insert(12, 18) << Move(11, 19, 5)) - << (SignalList() << Insert(5, 11) << Move(17, 18, 5) << Move(18, 19, 12)); - QTest::newRow("i(12-18),m(8-12,5)") - << (SignalList() << Insert(12, 18) << Move(8, 12, 5)) - << (SignalList() << Insert(12, 18) << Move(8, 12, 5)); - QTest::newRow("i(12-18),m(2-6,5)") - << (SignalList() << Insert(12, 18) << Move(2, 6, 5)) - << (SignalList() << Insert(12, 18) << Move(2, 6, 5)); - QTest::newRow("i(12-18),m(18-22,5)") - << (SignalList() << Insert(12, 18) << Move(18, 22, 5)) - << (SignalList() << Insert(12, 18) << Move(18, 22, 5)); - QTest::newRow("i(12-18),m(20-24,5)") - << (SignalList() << Insert(12, 18) << Move(20, 24, 5)) - << (SignalList() << Insert(12, 18) << Move(20, 24, 5)); - - QTest::newRow("i(12-18),m(5-13,11)") - << (SignalList() << Insert(12, 18) << Move(5, 13, 11)) - << (SignalList() << Insert(12, 17) << Insert(18, 19) << Move(5, 12, 11)); - - QTest::newRow("i(12-18),m(12-18,23)") - << (SignalList() << Insert(12, 18) << Move(12, 18, 23)) - << (SignalList() << Insert(23, 29)); - QTest::newRow("i(12-18),m(10-14,23)") - << (SignalList() << Insert(12, 18) << Move(10, 14, 23)) - << (SignalList() << Insert(12, 16) << Insert(25, 27) << Move(10, 12, 23)); - QTest::newRow("i(12-18),m(16-20,23)") - << (SignalList() << Insert(12, 18) << Move(16, 20, 23)) - << (SignalList() << Insert(12, 16) << Insert(25, 27) << Move(16, 18, 25)); - QTest::newRow("i(12-18),m(13-17,23)") - << (SignalList() << Insert(12, 18) << Move(13, 17, 23)) - << (SignalList() << Insert(12, 14) << Insert(23, 27)); - QTest::newRow("i(12-18),m(14-18,23)") - << (SignalList() << Insert(12, 18) << Move(14, 18, 23)) - << (SignalList() << Insert(12, 14) << Insert(23, 27)); - QTest::newRow("i(12-18),m(12-16,23)") - << (SignalList() << Insert(12, 18) << Move(12, 16, 23)) - << (SignalList() << Insert(12, 14) << Insert(23, 27)); - QTest::newRow("i(12-18),m(11-19,23)") - << (SignalList() << Insert(12, 18) << Move(11, 19, 23)) - << (SignalList() << Insert(25, 31) << Move(11, 12, 24) << Move(11, 12, 30)); - QTest::newRow("i(12-18),m(8-12,23)") - << (SignalList() << Insert(12, 18) << Move(8, 12, 23)) - << (SignalList() << Insert(12, 18) << Move(8, 12, 23)); - QTest::newRow("i(12-18),m(2-6,23)") - << (SignalList() << Insert(12, 18) << Move(2, 6, 23)) - << (SignalList() << Insert(12, 18) << Move(2, 6, 23)); - QTest::newRow("i(12-18),m(18-22,23)") - << (SignalList() << Insert(12, 18) << Move(18, 22, 23)) - << (SignalList() << Insert(12, 18) << Move(18, 22, 23)); - QTest::newRow("i(12-18),m(20-24,23)") - << (SignalList() << Insert(12, 18) << Move(20, 24, 23)) - << (SignalList() << Insert(12, 18) << Move(20, 24, 23)); - - QTest::newRow("i(12-18),m(11-21,23)") - << (SignalList() << Insert(12, 18) << Move(11, 21, 23)) - << (SignalList() << Insert(27, 33) << Move(11, 12, 26) << Move(11, 14, 30)); - - // Insert, then change - QTest::newRow("i(12-18),c(12-16)") - << (SignalList() << Insert(12, 18) << Change(12, 6)) - << (SignalList() << Insert(12, 18)); - QTest::newRow("i(12-18),c(10-14)") - << (SignalList() << Insert(12, 18) << Change(10, 16)) - << (SignalList() << Insert(12, 18) << Change(10, 12)); - QTest::newRow("i(12-18),c(16-20)") - << (SignalList() << Insert(12, 18) << Change(16, 20)) - << (SignalList() << Insert(12, 18) << Change(18, 20)); - QTest::newRow("i(12-18),c(13-17)") - << (SignalList() << Insert(12, 18) << Change(13, 17)) - << (SignalList() << Insert(12, 18)); - QTest::newRow("i(12-18),c(14-18)") - << (SignalList() << Insert(12, 18) << Change(14, 18)) - << (SignalList() << Insert(12, 18)); - QTest::newRow("i(12-18),c(12-16)") - << (SignalList() << Insert(12, 18) << Change(12, 16)) - << (SignalList() << Insert(12, 18)); - QTest::newRow("i(12-18),c(11-19)") - << (SignalList() << Insert(12, 18) << Change(11, 19)) - << (SignalList() << Insert(12, 18) << Change(11, 12) << Change(18, 19)); - QTest::newRow("i(12-18),c(8-12)") - << (SignalList() << Insert(12, 18) << Change(8, 12)) - << (SignalList() << Insert(12, 18) << Change(8, 12)); - QTest::newRow("i(12-18),c(2-6)") - << (SignalList() << Insert(12, 18) << Change(2, 6)) - << (SignalList() << Insert(12, 18) << Change(2, 6)); - QTest::newRow("i(12-18),c(18-22)") - << (SignalList() << Insert(12, 18) << Change(18, 22)) - << (SignalList() << Insert(12, 18) << Change(18, 22)); - QTest::newRow("i(12-18),c(20-24)") - << (SignalList() << Insert(12, 18) << Change(20, 24)) - << (SignalList() << Insert(12, 18) << Change(20, 24)); - - // Remove, then insert - QTest::newRow("r(12-18),i(12-18)") - << (SignalList() << Remove(12, 18) << Insert(12, 18)) - << (SignalList() << Remove(12, 18) << Insert(12, 18)); - QTest::newRow("r(12-18),i(10-14)") - << (SignalList() << Remove(12, 18) << Insert(10, 14)) - << (SignalList() << Remove(12, 18) << Insert(10, 14)); - QTest::newRow("r(12-18),i(16-20)") - << (SignalList() << Remove(12, 18) << Insert(16, 20)) - << (SignalList() << Remove(12, 18) << Insert(16, 20)); - QTest::newRow("r(12-18),i(13-17)") - << (SignalList() << Remove(12, 18) << Insert(13, 17)) - << (SignalList() << Remove(12, 18) << Insert(13, 17)); - QTest::newRow("r(12-18),i(14-18)") - << (SignalList() << Remove(12, 18) << Insert(14, 18)) - << (SignalList() << Remove(12, 18) << Insert(14, 18)); - QTest::newRow("r(12-18),i(12-16)") - << (SignalList() << Remove(12, 18) << Insert(12, 16)) - << (SignalList() << Remove(12, 18) << Insert(12, 16)); - QTest::newRow("r(12-18),i(11-19)") - << (SignalList() << Remove(12, 18) << Insert(11, 19)) - << (SignalList() << Remove(12, 18) << Insert(11, 19)); - QTest::newRow("i(12-18),r(8-12)") - << (SignalList() << Remove(12, 18) << Insert(8, 12)) - << (SignalList() << Remove(12, 18) << Insert(8, 12)); - QTest::newRow("i(12-18),r(2-6)") - << (SignalList() << Remove(12, 18) << Insert(2, 6)) - << (SignalList() << Remove(12, 18) << Insert(2, 6)); - QTest::newRow("i(12-18),r(18-22)") - << (SignalList() << Remove(12, 18) << Insert(18, 22)) - << (SignalList() << Remove(12, 18) << Insert(18, 22)); - QTest::newRow("i(12-18),r(20-24)") - << (SignalList() << Remove(12, 18) << Insert(20, 24)) - << (SignalList() << Remove(12, 18) << Insert(20, 24)); - - // Move, then insert - QTest::newRow("m(12-18,5),i(12-18)") - << (SignalList() << Move(12, 18, 5) << Insert(12, 18)) - << (SignalList() << Insert(6, 12) << Move(18, 24, 5)); - QTest::newRow("m(12-18,5),i(10-14)") - << (SignalList() << Move(12, 18, 5) << Insert(10, 14)) - << (SignalList() << Insert(5, 9) << Move(16, 21, 5) << Move(21, 22, 14)); - QTest::newRow("m(12-18,5),i(16-20)") - << (SignalList() << Move(12, 18, 5) << Insert(16, 20)) - << (SignalList() << Insert(10, 14) << Move(16, 22, 5)); - QTest::newRow("m(12-18,5),i(13-17)") - << (SignalList() << Move(12, 18, 5) << Insert(13, 17)) - << (SignalList() << Insert(7, 11) << Move(16, 22, 5)); - QTest::newRow("m(12-18,5),i(14-18)") - << (SignalList() << Move(12, 18, 5) << Insert(14, 18)) - << (SignalList() << Insert(8, 12) << Move(16, 22, 5)); - QTest::newRow("m(12-18,5),i(12-16)") - << (SignalList() << Move(12, 18, 5) << Insert(12, 16)) - << (SignalList() << Insert(6, 10) << Move(16, 22, 5)); - QTest::newRow("m(12-18,5),i(11-19)") - << (SignalList() << Move(12, 18, 5) << Insert(11, 19)) - << (SignalList() << Insert(5, 13) << Move(20, 26, 5)); - QTest::newRow("m(12-18,5),i(8-12)") - << (SignalList() << Move(12, 18, 5) << Insert(8, 12)) - << (SignalList() << Insert(5, 9) << Move(16, 19, 5) << Move(19, 22, 12)); - QTest::newRow("m(12-18,5),i(2-6)") - << (SignalList() << Move(12, 18, 5) << Insert(2, 6)) - << (SignalList() << Insert(2, 6) << Move(16, 22, 9)); - QTest::newRow("m(12-18,5),i(18-22)") - << (SignalList() << Move(12, 18, 5) << Insert(18, 22)) - << (SignalList() << Insert(18, 22) << Move(12, 18, 5)); - QTest::newRow("m(12-18,5),i(20-24)") - << (SignalList() << Move(12, 18, 5) << Insert(20, 24)) - << (SignalList() << Insert(20, 24) << Move(12, 18, 5)); - - QTest::newRow("m(12-18,23),i(12-18)") - << (SignalList() << Move(12, 18, 23) << Insert(12, 18)) - << (SignalList() << Insert(12, 18) << Move(18, 24, 29)); - QTest::newRow("m(12-18,23),i(10-14)") - << (SignalList() << Move(12, 18, 23) << Insert(10, 14)) - << (SignalList() << Insert(10, 14) << Move(16, 22, 27)); - QTest::newRow("m(12-18,23),i(16-20)") - << (SignalList() << Move(12, 18, 23) << Insert(16, 20)) - << (SignalList() << Insert(22, 26) << Move(12, 18, 27)); - QTest::newRow("m(12-18,23),i(13-17)") - << (SignalList() << Move(12, 18, 23) << Insert(13, 17)) - << (SignalList() << Insert(19, 23) << Move(12, 18, 27)); - QTest::newRow("m(12-18,23),i(14-18)") - << (SignalList() << Move(12, 18, 23) << Insert(14, 18)) - << (SignalList() << Insert(20, 24) << Move(12, 18, 27)); - QTest::newRow("m(12-18,23),i(12-16)") - << (SignalList() << Move(12, 18, 23) << Insert(12, 16)) - << (SignalList() << Insert(12, 16) << Move(16, 22, 27)); - QTest::newRow("m(12-18,23),i(11-19)") - << (SignalList() << Move(12, 18, 23) << Insert(11, 19)) - << (SignalList() << Insert(11, 19) << Move(20, 26, 31)); - QTest::newRow("m(12-18,23),i(8-12)") - << (SignalList() << Move(12, 18, 23) << Insert(8, 12)) - << (SignalList() << Insert(8, 12) << Move(16, 22, 27)); - QTest::newRow("m(12-18,23),i(2-6)") - << (SignalList() << Move(12, 18, 23) << Insert(2, 6)) - << (SignalList() << Insert(2, 6) << Move(16, 22, 27)); - QTest::newRow("m(12-18,23),i(18-22)") - << (SignalList() << Move(12, 18, 23) << Insert(18, 22)) - << (SignalList() << Insert(24, 28) << Move(12, 18, 27)); - QTest::newRow("m(12-18,23),i(20-24)") - << (SignalList() << Move(12, 18, 23) << Insert(20, 24)) - << (SignalList() << Insert(26, 30) << Move(12, 18, 27)); - - // Move, then remove - QTest::newRow("m(12-18,5),r(12-18)") - << (SignalList() << Move(12, 18, 5) << Remove(12, 18)) - << (SignalList() << Remove(6, 12) << Move(6, 12, 5)); - QTest::newRow("m(12-18,5),r(10-14)") - << (SignalList() << Move(12, 18, 5) << Remove(10, 14)) - << (SignalList() << Remove(5, 8) << Remove(14, 15) << Move(9, 14, 5)); - QTest::newRow("m(12-18,5),r(16-20)") - << (SignalList() << Move(12, 18, 5) << Remove(16, 20)) - << (SignalList() << Remove(10, 12) << Remove(16, 18) << Move(10, 16, 5)); - QTest::newRow("m(12-18,5),r(13-17)") - << (SignalList() << Move(12, 18, 5) << Remove(13, 17)) - << (SignalList() << Remove(7, 11) << Move(8, 14, 5)); - QTest::newRow("m(12-18,5),r(14-18)") - << (SignalList() << Move(12, 18, 5) << Remove(14, 18)) - << (SignalList() << Remove(8, 12) << Move(8, 14, 5)); - QTest::newRow("m(12-18,5),r(12-16)") - << (SignalList() << Move(12, 18, 5) << Remove(12, 16)) - << (SignalList() << Remove(6, 10) << Move(8, 14, 5)); - QTest::newRow("m(12-18,5),r(11-19)") - << (SignalList() << Move(12, 18, 5) << Remove(11, 19)) - << (SignalList() << Remove(5, 12) << Remove(11, 12)); - QTest::newRow("m(12-18,5),r(8-12)") - << (SignalList() << Move(12, 18, 5) << Remove(8, 12)) - << (SignalList() << Remove(5, 6) << Remove(14, 17) << Move(11, 14, 5)); - QTest::newRow("m(12-18,5),r(2-6)") - << (SignalList() << Move(12, 18, 5) << Remove(2, 6)) - << (SignalList() << Remove(2, 5) << Remove(9, 10) << Move(9, 14, 2)); - QTest::newRow("m(12-18,5),r(6-10)") - << (SignalList() << Move(12, 18, 5) << Remove(6, 10)) - << (SignalList() << Remove(13, 17) << Move(12, 14, 5)); - QTest::newRow("m(12-18,5),r(18-22)") - << (SignalList() << Move(12, 18, 5) << Remove(18, 22)) - << (SignalList() << Remove(18, 22) << Move(12, 18, 5)); - QTest::newRow("m(12-18,5),r(20-24)") - << (SignalList() << Move(12, 18, 5) << Remove(20, 24)) - << (SignalList() << Remove(20, 24) << Move(12, 18, 5)); - - QTest::newRow("m(12-18,23),r(12-18)") - << (SignalList() << Move(12, 18, 23) << Remove(12, 18)) - << (SignalList() << Remove(18, 24) << Move(12, 18, 17)); - QTest::newRow("m(12-18,23),r(10-14)") - << (SignalList() << Move(12, 18, 23) << Remove(10, 14)) - << (SignalList() << Remove(10, 12) << Remove(16, 18) << Move(10, 16, 19)); - QTest::newRow("m(12-18,23),r(16-20)") - << (SignalList() << Move(12, 18, 23) << Remove(16, 20)) - << (SignalList() << Remove(22, 26) << Move(12, 18, 19)); - QTest::newRow("m(12-18,23),r(13-17)") - << (SignalList() << Move(12, 18, 23) << Remove(13, 17)) - << (SignalList() << Remove(19, 23) << Move(12, 18, 19)); - QTest::newRow("m(12-18,23),r(14-18)") - << (SignalList() << Move(12, 18, 23) << Remove(14, 18)) - << (SignalList() << Remove(20, 24) << Move(12, 18, 19)); - QTest::newRow("m(12-18,23),r(12-16)") - << (SignalList() << Move(12, 18, 23) << Remove(12, 16)) - << (SignalList() << Remove(18, 22) << Move(12, 18, 19)); - QTest::newRow("m(12-18,23),r(11-19)") - << (SignalList() << Move(12, 18, 23) << Remove(11, 19)) - << (SignalList() << Remove(11, 12) << Remove(17, 24) << Move(11, 17, 15)); - QTest::newRow("m(12-18,23),r(8-12)") - << (SignalList() << Move(12, 18, 23) << Remove(8, 12)) - << (SignalList() << Remove(8, 12) << Move(8, 14, 19)); - QTest::newRow("m(12-18,23),r(2-6)") - << (SignalList() << Move(12, 18, 23) << Remove(2, 6)) - << (SignalList() << Remove(2, 6) << Move(8, 14, 19)); - QTest::newRow("m(12-18,23),r(18-22)") - << (SignalList() << Move(12, 18, 23) << Remove(18, 22)) - << (SignalList() << Remove(24, 28) << Move(12, 18, 19)); - QTest::newRow("m(12-18,23),r(20-24)") - << (SignalList() << Move(12, 18, 23) << Remove(20, 24)) - << (SignalList() << Remove(12, 13) << Remove(25, 28) << Move(12, 17, 20)); + QTest::newRow("i(12,6),r(10,4)") + << (SignalList() << Insert(12,6) << Remove(10,4)) + << (SignalList() << Remove(10,2) << Insert(10,4)); + QTest::newRow("i(12,6),r(16,4)") + << (SignalList() << Insert(12,6) << Remove(16,4)) + << (SignalList() << Remove(12,2) << Insert(12,4)); + QTest::newRow("i(12,6),r(13,4)") + << (SignalList() << Insert(12,6) << Remove(13,4)) + << (SignalList() << Insert(12,2)); + QTest::newRow("i(12,6),r(14,4)") + << (SignalList() << Insert(12,6) << Remove(14,4)) + << (SignalList() << Insert(12,2)); + QTest::newRow("i(12,6),r(12,4)") + << (SignalList() << Insert(12,6) << Remove(12,4)) + << (SignalList() << Insert(12,2)); + QTest::newRow("i(12,6),r(11,8)") + << (SignalList() << Insert(12,6) << Remove(11,8)) + << (SignalList() << Remove(11,2)); + QTest::newRow("i(12,6),r(8,4)") + << (SignalList() << Insert(12,6) << Remove(8,4)) + << (SignalList() << Remove(8,4) << Insert(8,6)); + QTest::newRow("i(12,6),r(2,4)") + << (SignalList() << Insert(12,6) << Remove(2,4)) + << (SignalList() << Remove(2,4) << Insert(8,6)); + QTest::newRow("i(12,6),r(18,4)") + << (SignalList() << Insert(12,6) << Remove(18,4)) + << (SignalList() << Remove(12,4) << Insert(12,6)); + QTest::newRow("i(12,6),r(20,4)") + << (SignalList() << Insert(12,6) << Remove(20,4)) + << (SignalList() << Remove(14,4) << Insert(12,6)); + + // Insert,then move + QTest::newRow("i(12,6),m(12-5,6)") + << (SignalList() << Insert(12,6) << Move(12,5,6)) + << (SignalList() << Insert(5,6)); + QTest::newRow("i(12,6),m(10-5,4)") + << (SignalList() << Insert(12,6) << Move(10,5,4)) + << (SignalList() << Remove(10,2,0) << Insert(5,2,0) << Insert(7,2) << Insert(14,4)); + QTest::newRow("i(12,6),m(16-5,4)") + << (SignalList() << Insert(12,6) << Move(16,5,4)) + << (SignalList() << Remove(12,2,0) << Insert(5,2) << Insert(7,2,0) << Insert(16,4)); + QTest::newRow("i(12,6),m(13-5,4)") + << (SignalList() << Insert(12,6) << Move(13,5,4)) + << (SignalList() << Insert(5,4) << Insert(16,2)); + QTest::newRow("i(12,6),m(14-5,4)") + << (SignalList() << Insert(12,6) << Move(14,5,4)) + << (SignalList() << Insert(5,4) << Insert(16,2)); + QTest::newRow("i(12,6),m(12-5,4)") + << (SignalList() << Insert(12,6) << Move(12,5,4)) + << (SignalList() << Insert(5,4) << Insert(16,2)); + QTest::newRow("i(12,6),m(11-5,8)") + << (SignalList() << Insert(12,6) << Move(11,5,8)) + << (SignalList() << Remove(11,1,0) << Remove(11,1,1) << Insert(5,1,0) << Insert(6,6) << Insert(12,1,1)); + QTest::newRow("i(12,6),m(8-5,4)") + << (SignalList() << Insert(12,6) << Move(8,5,4)) + << (SignalList() << Remove(8,4,0) << Insert(5,4,0) << Insert(12,6)); + QTest::newRow("i(12,6),m(2-5,4)") + << (SignalList() << Insert(12,6) << Move(2,5,4)) + << (SignalList() << Remove(2,4,0) << Insert(5,4,0) << Insert(12,6)); + QTest::newRow("i(12,6),m(18-5,4)") + << (SignalList() << Insert(12,6) << Move(18,5,4)) + << (SignalList() << Remove(12,4,0) << Insert(5,4,0) << Insert(16,6)); + QTest::newRow("i(12,6),m(20-5,4)") + << (SignalList() << Insert(12,6) << Move(20,5,4)) + << (SignalList() << Remove(14,4,0) << Insert(5,4,0) << Insert(16,6)); + + QTest::newRow("i(12,6),m(5-13,11)") + << (SignalList() << Insert(12,6) << Move(5,11,8)) + << (SignalList() << Remove(5,7,0) << Insert(5,5) << Insert(11,7,0) << Insert(18,1)); + + QTest::newRow("i(12,6),m(12-23,6)") + << (SignalList() << Insert(12,6) << Move(12,23,6)) + << (SignalList() << Insert(23,6)); + QTest::newRow("i(12,6),m(10-23,4)") + << (SignalList() << Insert(12,6) << Move(10,23,4)) + << (SignalList() << Remove(10,2,0) << Insert(10,4) << Insert(23,2,0) << Insert(25,2)); + QTest::newRow("i(12,6),m(16-23,4)") + << (SignalList() << Insert(12,6) << Move(16,23,4)) + << (SignalList() << Remove(12,2,0) << Insert(12,4) << Insert(23,2) << Insert(25,2,0)); + QTest::newRow("i(12,6),m(13-23,4)") + << (SignalList() << Insert(12,6) << Move(13,23,4)) + << (SignalList() << Insert(12,2) << Insert(23,4)); + QTest::newRow("i(12,6),m(14-23,4)") + << (SignalList() << Insert(12,6) << Move(14,23,4)) + << (SignalList() << Insert(12,2) << Insert(23,4)); + QTest::newRow("i(12,6),m(12-23,4)") + << (SignalList() << Insert(12,6) << Move(12,23,4)) + << (SignalList() << Insert(12,2) << Insert(23,4)); + QTest::newRow("i(12,6),m(11-23,8)") + << (SignalList() << Insert(12,6) << Move(11,23,8)) + << (SignalList() << Remove(11,1,0) << Remove(11,1,1) << Insert(23,1,0)<< Insert(24,6) << Insert(30,1,1)); + QTest::newRow("i(12,6),m(8-23,4)") + << (SignalList() << Insert(12,6) << Move(8,23,4)) + << (SignalList() << Remove(8,4,0) << Insert(8,6) << Insert(23,4,0)); + QTest::newRow("i(12,6),m(2-23,4)") + << (SignalList() << Insert(12,6) << Move(2,23,4)) + << (SignalList() << Remove(2,4,0) << Insert(8,6) << Insert(23,4,0)); + QTest::newRow("i(12,6),m(18-23,4)") + << (SignalList() << Insert(12,6) << Move(18,23,4)) + << (SignalList() << Remove(12,4,0) << Insert(12,6) << Insert(23,4,0)); + QTest::newRow("i(12,6),m(20-23,4)") + << (SignalList() << Insert(12,6) << Move(20,23,4)) + << (SignalList() << Remove(14,4,0) << Insert(12,6) << Insert(23,4,0)); + + QTest::newRow("i(12,6),m(11-23,10)") + << (SignalList() << Insert(12,6) << Move(11,23,10)) + << (SignalList() << Remove(11,1,0) << Remove(11,3,1) << Insert(23,1,0) << Insert(24,6) << Insert(30,3,1)); + + // Insert,then change + QTest::newRow("i(12,6),c(12,6)") + << (SignalList() << Insert(12,6) << Change(12,6)) + << (SignalList() << Insert(12,6)); + QTest::newRow("i(12,6),c(10,6)") + << (SignalList() << Insert(12,6) << Change(10,6)) + << (SignalList() << Insert(12,6) << Change(10,2)); + QTest::newRow("i(12,6),c(16,4)") + << (SignalList() << Insert(12,6) << Change(16,4)) + << (SignalList() << Insert(12,6) << Change(18,2)); + QTest::newRow("i(12,6),c(13,4)") + << (SignalList() << Insert(12,6) << Change(13,4)) + << (SignalList() << Insert(12,6)); + QTest::newRow("i(12,6),c(14,4)") + << (SignalList() << Insert(12,6) << Change(14,4)) + << (SignalList() << Insert(12,6)); + QTest::newRow("i(12,6),c(12,4)") + << (SignalList() << Insert(12,6) << Change(12,4)) + << (SignalList() << Insert(12,6)); + QTest::newRow("i(12,6),c(11,8)") + << (SignalList() << Insert(12,6) << Change(11,8)) + << (SignalList() << Insert(12,6) << Change(11,1) << Change(18,1)); + QTest::newRow("i(12,6),c(8,4)") + << (SignalList() << Insert(12,6) << Change(8,4)) + << (SignalList() << Insert(12,6) << Change(8,4)); + QTest::newRow("i(12,6),c(2,4)") + << (SignalList() << Insert(12,6) << Change(2,4)) + << (SignalList() << Insert(12,6) << Change(2,4)); + QTest::newRow("i(12,6),c(18,4)") + << (SignalList() << Insert(12,6) << Change(18,4)) + << (SignalList() << Insert(12,6) << Change(18,4)); + QTest::newRow("i(12,6),c(20,4)") + << (SignalList() << Insert(12,6) << Change(20,4)) + << (SignalList() << Insert(12,6) << Change(20,4)); + + // Remove,then insert + QTest::newRow("r(12,6),i(12,6)") + << (SignalList() << Remove(12,6) << Insert(12,6)) + << (SignalList() << Remove(12,6) << Insert(12,6)); + QTest::newRow("r(12,6),i(10,4)") + << (SignalList() << Remove(12,6) << Insert(10,14)) + << (SignalList() << Remove(12,6) << Insert(10,14)); + QTest::newRow("r(12,6),i(16,4)") + << (SignalList() << Remove(12,6) << Insert(16,4)) + << (SignalList() << Remove(12,6) << Insert(16,4)); + QTest::newRow("r(12,6),i(13,4)") + << (SignalList() << Remove(12,6) << Insert(13,4)) + << (SignalList() << Remove(12,6) << Insert(13,4)); + QTest::newRow("r(12,6),i(14,4)") + << (SignalList() << Remove(12,6) << Insert(14,4)) + << (SignalList() << Remove(12,6) << Insert(14,4)); + QTest::newRow("r(12,6),i(12,4)") + << (SignalList() << Remove(12,6) << Insert(12,4)) + << (SignalList() << Remove(12,6) << Insert(12,4)); + QTest::newRow("r(12,6),i(11,8)") + << (SignalList() << Remove(12,6) << Insert(11,8)) + << (SignalList() << Remove(12,6) << Insert(11,8)); + QTest::newRow("r(12,6),i(8,4)") + << (SignalList() << Remove(12,6) << Insert(8,4)) + << (SignalList() << Remove(12,6) << Insert(8,4)); + QTest::newRow("r(12,6),i(2,4)") + << (SignalList() << Remove(12,6) << Insert(2,4)) + << (SignalList() << Remove(12,6) << Insert(2,4)); + QTest::newRow("r(12,6),i(18,4)") + << (SignalList() << Remove(12,6) << Insert(18,4)) + << (SignalList() << Remove(12,6) << Insert(18,4)); + QTest::newRow("r(12,6),i(20,4)") + << (SignalList() << Remove(12,6) << Insert(20,4)) + << (SignalList() << Remove(12,6) << Insert(20,4)); + + // Move,then insert + QTest::newRow("m(12-5,6),i(12,6)") + << (SignalList() << Move(12,5,6) << Insert(12,6)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(12,6)); + QTest::newRow("m(12-5,6),i(10,4)") + << (SignalList() << Move(12,5,6) << Insert(10,4)) + << (SignalList() << Remove(12,5,0) << Remove(12,1,1) << Insert(5,5,0) << Insert(10,4) << Insert(14,1,1)); + QTest::newRow("m(12-5,6),i(16,4)") + << (SignalList() << Move(12,5,6) << Insert(16,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(16,4)); + QTest::newRow("m(12-5,6),i(13,4)") + << (SignalList() << Move(12,5,6) << Insert(13,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(13,4)); + QTest::newRow("m(12-5,6),i(14,4)") + << (SignalList() << Move(12,5,6) << Insert(14,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(14,4)); + QTest::newRow("m(12-5,6),i(12,4)") + << (SignalList() << Move(12,5,6) << Insert(12,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(12,4)); + QTest::newRow("m(12-5,6),i(11,8)") + << (SignalList() << Move(12,5,6) << Insert(11,8)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(11,8)); + QTest::newRow("m(12-5,6),i(8,4)") + << (SignalList() << Move(12,5,6) << Insert(8,4)) + << (SignalList() << Remove(12,3,0) << Remove(12,3,1) << Insert(5,3,0) << Insert(8,4) << Insert(12,3,1)); + QTest::newRow("m(12-5,6),i(2,4)") + << (SignalList() << Move(12,5,6) << Insert(2,4)) + << (SignalList() << Remove(12,6,0) << Insert(2,4) << Insert(9,6,0)); + QTest::newRow("m(12-5,6),i(18,4)") + << (SignalList() << Move(12,5,6) << Insert(18,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(18,4)); + QTest::newRow("m(12-5,6),i(20,4)") + << (SignalList() << Move(12,5,6) << Insert(20,4)) + << (SignalList() << Remove(12,6,0) << Insert(5,6,0) << Insert(20,4)); + + QTest::newRow("m(12-23,6),i(12,6)") + << (SignalList() << Move(12,23,6) << Insert(12,6)) + << (SignalList() << Remove(12,6,0) << Insert(12,6) << Insert(29,6,0)); + QTest::newRow("m(12-23,6),i(10,4)") + << (SignalList() << Move(12,23,6) << Insert(10,4)) + << (SignalList() << Remove(12,6,0) << Insert(10,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(16,4)") + << (SignalList() << Move(12,23,6) << Insert(16,4)) + << (SignalList() << Remove(12,6,0) << Insert(16,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(13,4)") + << (SignalList() << Move(12,23,6) << Insert(13,4)) + << (SignalList() << Remove(12,6,0) << Insert(13,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(14,4)") + << (SignalList() << Move(12,23,6) << Insert(14,4)) + << (SignalList() << Remove(12,6,0) << Insert(14,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(12,4)") + << (SignalList() << Move(12,23,6) << Insert(12,4)) + << (SignalList() << Remove(12,6,0) << Insert(12,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(11,8)") + << (SignalList() << Move(12,23,6) << Insert(11,8)) + << (SignalList() << Remove(12,6,0) << Insert(11,8) << Insert(31,6,0)); + QTest::newRow("m(12-23,6),i(8,4)") + << (SignalList() << Move(12,23,6) << Insert(8,4)) + << (SignalList() << Remove(12,6,0) << Insert(8,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(2,4)") + << (SignalList() << Move(12,23,6) << Insert(2,4)) + << (SignalList() << Remove(12,6,0) << Insert(2,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(18,4)") + << (SignalList() << Move(12,23,6) << Insert(18,4)) + << (SignalList() << Remove(12,6,0) << Insert(18,4) << Insert(27,6,0)); + QTest::newRow("m(12-23,6),i(20,4)") + << (SignalList() << Move(12,23,6) << Insert(20,4)) + << (SignalList() << Remove(12,6,0) << Insert(20,4) << Insert(27,6,0)); + + // Move,then remove + QTest::newRow("m(12-5,6),r(12,6)") + << (SignalList() << Move(12,5,6) << Remove(12,6)) + << (SignalList() << Remove(6,6) << Remove(6,6,0) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(10,4)") + << (SignalList() << Move(12,5,6) << Remove(10,4)) + << (SignalList() << Remove(5,3) << Remove(9,5,0) << Remove(9,1) << Insert(5,5,0)); + QTest::newRow("m(12-5,6),r(16,4)") + << (SignalList() << Move(12,5,6) << Remove(16,4)) + << (SignalList() << Remove(10,2) << Remove(10,6,0) << Remove(10,2) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(13,4)") + << (SignalList() << Move(12,5,6) << Remove(13,4)) + << (SignalList() << Remove(7,4) << Remove(8,6,0) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(14,4)") + << (SignalList() << Move(12,5,6) << Remove(14,4)) + << (SignalList() << Remove(8,4) << Remove(8,6,0) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(12,4)") + << (SignalList() << Move(12,5,6) << Remove(12,4)) + << (SignalList() << Remove(6,4) << Remove(8,6,0) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(11,8)") + << (SignalList() << Move(12,5,6) << Remove(11,8)) + << (SignalList() << Remove(5,7) << Remove(5,6,0) << Remove(5,1) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(8,4)") + << (SignalList() << Move(12,5,6) << Remove(8,4)) + << (SignalList() << Remove(5,1) << Remove(11,3,0) << Remove(11,3) << Insert(5,3,0)); + QTest::newRow("m(12-5,6),r(2,4)") + << (SignalList() << Move(12,5,6) << Remove(2,4)) + << (SignalList() << Remove(2,3) << Remove(9,1) << Remove(9,5,0) << Insert(2,5,0)); + QTest::newRow("m(12-5,6),r(6,4)") + << (SignalList() << Move(12,5,6) << Remove(6,4)) + << (SignalList() << Remove(12,1,0) << Remove(12,4) << Remove(12,1,1) << Insert(5,1,0) << Insert(6,1,1)); + QTest::newRow("m(12-5,6),r(18,4)") + << (SignalList() << Move(12,5,6) << Remove(18,4)) + << (SignalList() << Remove(12,6,0) << Remove(12,4) << Insert(5,6,0)); + QTest::newRow("m(12-5,6),r(20,4)") + << (SignalList() << Move(12,5,6) << Remove(20,4)) + << (SignalList() << Remove(12,6,0) << Remove(14,4) << Insert(5,6,0)); + + QTest::newRow("m(12-23,6),r(12,6)") + << (SignalList() << Move(12,23,6) << Remove(12,6)) + << (SignalList() << Remove(12,6,0) << Remove(12,6) << Insert(17,6,0)); + QTest::newRow("m(12-23,6),r(10,4)") + << (SignalList() << Move(12,23,6) << Remove(10,4)) + << (SignalList() << Remove(10,2) << Remove(10,6,0) << Remove(10,2) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(16,4)") + << (SignalList() << Move(12,23,6) << Remove(16,4)) + << (SignalList() << Remove(12,6,0) << Remove(16,4) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(13,4)") + << (SignalList() << Move(12,23,6) << Remove(13,4)) + << (SignalList() << Remove(12,6,0) << Remove(13,4) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(14,4)") + << (SignalList() << Move(12,23,6) << Remove(14,4)) + << (SignalList() << Remove(12,6,0) << Remove(14,4) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(12,4)") + << (SignalList() << Move(12,23,6) << Remove(12,4)) + << (SignalList() << Remove(12,6,0) << Remove(12,4) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(11,8)") + << (SignalList() << Move(12,23,6) << Remove(11,8)) + << (SignalList() << Remove(11,1) << Remove(11,6,0) << Remove(11,7) << Insert(15,6,0)); + QTest::newRow("m(12-23,6),r(8,4)") + << (SignalList() << Move(12,23,6) << Remove(8,4)) + << (SignalList() << Remove(8,4) << Remove(8,6,0) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(2,4)") + << (SignalList() << Move(12,23,6) << Remove(2,4)) + << (SignalList() << Remove(2,4) << Remove(8,6,0) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(18,4)") + << (SignalList() << Move(12,23,6) << Remove(18,4)) + << (SignalList() << Remove(12,6,0) << Remove(18,4) << Insert(19,6,0)); + QTest::newRow("m(12-23,6),r(20,4)") + << (SignalList() << Move(12,23,6) << Remove(20,4)) + << (SignalList() << Remove(12,1) << Remove(12,5,0) << Remove(20,3) << Insert(20,5,0)); } void tst_qdeclarativemodelchange::sequence() @@ -560,24 +733,22 @@ void tst_qdeclarativemodelchange::sequence() foreach (const Signal &signal, input) { if (signal.isRemove()) - set.insertRemove(signal.start, signal.end); + set.remove(signal.index, signal.count); else if (signal.isInsert()) - set.insertInsert(signal.start, signal.end); + set.insert(signal.index, signal.count); else if (signal.isMove()) - set.insertMove(signal.start, signal.end, signal.to); + set.move(signal.index, signal.to, signal.count); else if (signal.isChange()) - set.insertChange(signal.start, signal.end); + set.change(signal.index, signal.count); } SignalList changes; foreach (const QDeclarativeChangeSet::Remove &remove, set.removes()) - changes << Remove(remove.start, remove.end); + changes << Remove(remove.index, remove.count, remove.moveId); foreach (const QDeclarativeChangeSet::Insert &insert, set.inserts()) - changes << Insert(insert.start, insert.end); - foreach (const QDeclarativeChangeSet::Move &move, set.moves()) - changes << Move(move.start, move.end, move.to); + changes << Insert(insert.index, insert.count, insert.moveId); foreach (const QDeclarativeChangeSet::Change &change, set.changes()) - changes << Change(change.start, change.end); + changes << Change(change.index, change.count); #ifdef VERIFY_EXPECTED_OUTPUT QVector list;