Revert "Revert "Editor/LayersEditor: Simplify moveRows implementation""

This reverts commit 4fdb2bde8c.
This commit is contained in:
Lioncash 2019-08-26 22:40:14 -04:00
parent 8569cab4d7
commit a178d55007
1 changed files with 26 additions and 23 deletions

View File

@ -1,8 +1,13 @@
#include "LayersEditor.hpp"
#include "MainWindow.hpp"
#include <QVBoxLayout>
#include <QScrollBar>
#include <algorithm>
#include <QMimeData>
#include <QScrollBar>
#include <QVBoxLayout>
#include "MainWindow.hpp"
class LayerDataChangeUndoCommand : public EditorUndoCommand {
QModelIndex m_index;
@ -446,29 +451,27 @@ bool LayersModel::insertRows(int row, int count, const QModelIndex& parent) {
bool LayersModel::moveRows(const QModelIndex& sourceParent, int sourceRow, int count,
const QModelIndex& destinationParent, int destinationChild) {
if (!m_node)
if (!m_node) {
return false;
bool moving = beginMoveRows(sourceParent, sourceRow, sourceRow + count - 1, destinationParent, destinationChild);
std::vector<amuse::LayerMapping>& layers = *m_node->m_obj;
if (destinationChild < sourceRow) {
for (int i = 0; i < count; ++i) {
amuse::LayerMapping tmp = std::move(layers[sourceRow]);
for (int j = sourceRow; j != destinationChild; --j)
layers[j] = std::move(layers[j - 1]);
layers[destinationChild] = std::move(tmp);
++sourceRow;
++destinationChild;
}
} else if (destinationChild > sourceRow) {
for (int i = 0; i < count; ++i) {
amuse::LayerMapping tmp = std::move(layers[sourceRow]);
for (int j = sourceRow; j != destinationChild - 1; ++j)
layers[j] = std::move(layers[j + 1]);
layers[destinationChild - 1] = std::move(tmp);
}
}
if (moving)
const bool moving =
beginMoveRows(sourceParent, sourceRow, sourceRow + count - 1, destinationParent, destinationChild);
std::vector<amuse::LayerMapping>& layers = *m_node->m_obj;
const auto pivot = std::make_move_iterator(layers.begin() + sourceRow);
const auto begin = std::make_move_iterator(layers.begin() + destinationChild);
const auto end = pivot + count;
if (destinationChild < sourceRow) {
std::rotate(begin, pivot, end);
} else if (destinationChild > sourceRow) {
std::rotate(pivot, end, begin);
}
if (moving) {
endMoveRows();
}
return true;
}