diff options
author | Richard Moe Gustavsen <richard.gustavsen@qt.io> | 2018-08-10 14:25:22 +0200 |
---|---|---|
committer | Richard Moe Gustavsen <richard.gustavsen@qt.io> | 2018-08-10 15:00:19 +0000 |
commit | 0bd3c9cda678b7e158c6142c1567ac927fde18b9 (patch) | |
tree | 551b1e91b2365c1674f14fdec4bc7a2df7cde4c9 /src | |
parent | 481ea25bc8aa64e7131edd85a6c871b2a3193f58 (diff) |
QQuickTableView: remove cacheBuffer from the public API
As discussed during API review, remove cacheBuffer from the public API.
The cache buffer was a feature inherited from ListView to avoid loading
a lot of items (and affect performance) when the user started to flick.
But now that TableView has support for reusing items, the point of the
cache buffer is more or less gone. At least we choose to remove it from
the public API until we have better understanding if this is really needed.
Note that the cacheBuffer still plays a small role internally, so we
don't remove it from the implementation. We want to preload an extra row
and column for reuse at start-up, since you often cannot reuse the first
row and column during the first flick (they will still be visible
on the screen).
Change-Id: Ie62835a04ac29a84c9a76151b73fe2f75d9ae844
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/quick/items/qquicktableview.cpp | 20 | ||||
-rw-r--r-- | src/quick/items/qquicktableview_p.h | 5 |
2 files changed, 0 insertions, 25 deletions
diff --git a/src/quick/items/qquicktableview.cpp b/src/quick/items/qquicktableview.cpp index d308621bc6..6a7bc78c4d 100644 --- a/src/quick/items/qquicktableview.cpp +++ b/src/quick/items/qquicktableview.cpp @@ -1446,26 +1446,6 @@ void QQuickTableView::setRightMargin(qreal margin) emit rightMarginChanged(); } -int QQuickTableView::cacheBuffer() const -{ - return d_func()->cacheBuffer; -} - -void QQuickTableView::setCacheBuffer(int newBuffer) -{ - Q_D(QQuickTableView); - if (d->cacheBuffer == newBuffer || newBuffer < 0) - return; - - d->cacheBuffer = newBuffer; - - if (newBuffer == 0) - d->unloadBuffer(); - - emit cacheBufferChanged(); - polish(); -} - QJSValue QQuickTableView::rowHeightProvider() const { return d_func()->rowHeightProvider; diff --git a/src/quick/items/qquicktableview_p.h b/src/quick/items/qquicktableview_p.h index ff9a22f9af..1ad74fa438 100644 --- a/src/quick/items/qquicktableview_p.h +++ b/src/quick/items/qquicktableview_p.h @@ -73,7 +73,6 @@ class Q_QUICK_PRIVATE_EXPORT QQuickTableView : public QQuickFlickable Q_PROPERTY(qreal bottomMargin READ bottomMargin WRITE setBottomMargin NOTIFY bottomMarginChanged) Q_PROPERTY(qreal leftMargin READ leftMargin WRITE setLeftMargin NOTIFY leftMarginChanged) Q_PROPERTY(qreal rightMargin READ rightMargin WRITE setRightMargin NOTIFY rightMarginChanged) - Q_PROPERTY(int cacheBuffer READ cacheBuffer WRITE setCacheBuffer NOTIFY cacheBufferChanged) Q_PROPERTY(QJSValue rowHeightProvider READ rowHeightProvider WRITE setRowHeightProvider NOTIFY rowHeightProviderChanged) Q_PROPERTY(QJSValue columnWidthProvider READ columnWidthProvider WRITE setColumnWidthProvider NOTIFY columnWidthProviderChanged) Q_PROPERTY(QVariant model READ model WRITE setModel NOTIFY modelChanged) @@ -106,9 +105,6 @@ public: qreal rightMargin() const; void setRightMargin(qreal margin); - int cacheBuffer() const; - void setCacheBuffer(int newBuffer); - QJSValue rowHeightProvider() const; void setRowHeightProvider(QJSValue provider); @@ -142,7 +138,6 @@ Q_SIGNALS: void bottomMarginChanged(); void leftMarginChanged(); void rightMarginChanged(); - void cacheBufferChanged(); void rowHeightProviderChanged(); void columnWidthProviderChanged(); void modelChanged(); |