From 8462a6933c843eeed2d94d83d4b10f7c946a5475 Mon Sep 17 00:00:00 2001 From: Giuseppe D'Angelo Date: Fri, 8 Nov 2013 13:22:36 +0100 Subject: Remove a boolean trap in QLayout::replaceWidget MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use some flags instead. Task-number: QTBUG-34668 Change-Id: I9a75253c8eb98164c594bb6bb06c1a16c9609537 Reviewed-by: Thorbjørn Lund Martsum Reviewed-by: J-P Nurmi Reviewed-by: Marc Mutz --- src/widgets/kernel/qlayout.cpp | 24 ++++++++++++++++-------- 1 file changed, 16 insertions(+), 8 deletions(-) (limited to 'src/widgets/kernel/qlayout.cpp') diff --git a/src/widgets/kernel/qlayout.cpp b/src/widgets/kernel/qlayout.cpp index f17a6caa3c..eb21a580b1 100644 --- a/src/widgets/kernel/qlayout.cpp +++ b/src/widgets/kernel/qlayout.cpp @@ -1112,18 +1112,26 @@ bool QLayout::activate() \since 5.2 Searches for widget \a from and replaces it with widget \a to if found. - Returns the layout item that contains the widget \a from on success. Otherwise \c 0 is returned. - If \a recursive is \c true, sub-layouts are searched for doing the replacement. Notice that the returned item therefore might not belong to this layout, but to a sub-layout. + Returns the layout item that contains the widget \a from on success. + Otherwise \c 0 is returned. If \a options contains \c Qt::FindChildrenRecursively + (the default), sub-layouts are searched for doing the replacement. + Any other flag in \a options is ignored. - The returned layout item is no longer owned by the layout and should be either deleted or inserted to another layout. The widget \a from is no longer managed by the layout and may need to be deleted or hidden. The parent of widget \a from is left unchanged. + Notice that the returned item therefore might not belong to this layout, + but to a sub-layout. - This function works for the built-in Qt layouts, but might not work for custom layouts. + The returned layout item is no longer owned by the layout and should be + either deleted or inserted to another layout. The widget \a from is no + longer managed by the layout and may need to be deleted or hidden. The + parent of widget \a from is left unchanged. + + This function works for the built-in Qt layouts, but might not work for + custom layouts. \sa indexOf() */ -//### Qt 6 make this function virtual -QLayoutItem* QLayout::replaceWidget(QWidget *from, QWidget *to, bool recursive) +QLayoutItem *QLayout::replaceWidget(QWidget *from, QWidget *to, Qt::FindChildOptions options) { Q_D(QLayout); if (!from || !to) @@ -1141,8 +1149,8 @@ QLayoutItem* QLayout::replaceWidget(QWidget *from, QWidget *to, bool recursive) break; } - if (item->layout() && recursive) { - QLayoutItem *r = item->layout()->replaceWidget(from, to, true); + if (item->layout() && (options & Qt::FindChildrenRecursively)) { + QLayoutItem *r = item->layout()->replaceWidget(from, to, options); if (r) return r; } -- cgit v1.2.3