summaryrefslogtreecommitdiff
path: root/src/layouts/qquicklinearlayout.cpp
diff options
context:
space:
mode:
authorJan Arve Saether <jan-arve.saether@digia.com>2013-04-10 10:09:22 +0200
committerThe Qt Project <gerrit-noreply@qt-project.org>2013-04-10 10:34:28 +0200
commit628d3eeb4b91ef0159a87f3add5d97a77af5fec6 (patch)
tree48b5c24e940f77f4d2d2e715389dc0b58e4c440b /src/layouts/qquicklinearlayout.cpp
parent6a220dfc1a363d85f486d7de04834e501b60b67f (diff)
downloadqtquickcontrols-628d3eeb4b91ef0159a87f3add5d97a77af5fec6.tar.gz
Remove a function that doesn't serve any purpose anymore.
setupItemLayout was supposed to do initialization on each item before it got added to the layout, but all it did was to add the attached property to the item, which is really not needed anymore. The layout code does not work under the assumption that each item has the attached property anyway, so we can remove it. (And in addition avoid some potential allocations) Change-Id: I6d7e3e5b13531dfeaceef495de144dcf87d38cc4 Reviewed-by: Jens Bache-Wiig <jens.bache-wiig@digia.com>
Diffstat (limited to 'src/layouts/qquicklinearlayout.cpp')
-rw-r--r--src/layouts/qquicklinearlayout.cpp4
1 files changed, 0 insertions, 4 deletions
diff --git a/src/layouts/qquicklinearlayout.cpp b/src/layouts/qquicklinearlayout.cpp
index 4fe50b4d..c452e4fd 100644
--- a/src/layouts/qquicklinearlayout.cpp
+++ b/src/layouts/qquicklinearlayout.cpp
@@ -498,8 +498,6 @@ void QQuickGridLayout::insertLayoutItems()
QQuickGridLayoutItem *layoutItem = new QQuickGridLayoutItem(child, row, column, rowSpan, columnSpan);
d->engine.insertItem(layoutItem, -1);
-
- setupItemLayout(child);
}
}
}
@@ -550,8 +548,6 @@ void QQuickLinearLayout::insertLayoutItems()
qSwap(gridRow, gridColumn);
QQuickGridLayoutItem *layoutItem = new QQuickGridLayoutItem(child, gridRow, gridColumn, 1, 1, 0);
d->engine.insertItem(layoutItem, index);
-
- setupItemLayout(child);
}
}
}