1 files changed, 31 insertions, 46 deletions
diff --git a/src/widgets/graphicsview/qgraphicslayoutitem.cpp b/src/widgets/graphicsview/qgraphicslayoutitem.cpp
index aed154a95f..f6aad09810 100644
--- a/src/widgets/graphicsview/qgraphicslayoutitem.cpp
+++ b/src/widgets/graphicsview/qgraphicslayoutitem.cpp
@@ -1,41 +1,5 @@
-/****************************************************************************
-**
-** Copyright (C) 2016 The Qt Company Ltd.
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of the QtWidgets module of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:LGPL$
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU Lesser General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU Lesser
-** General Public License version 3 as published by the Free Software
-** Foundation and appearing in the file LICENSE.LGPL3 included in the
-** packaging of this file. Please review the following information to
-** ensure the GNU Lesser General Public License version 3 requirements
-** will be met: https://www.gnu.org/licenses/lgpl-3.0.html.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 2.0 or (at your option) the GNU General
-** Public license version 3 or any later version approved by the KDE Free
-** Qt Foundation. The licenses are as published by the Free Software
-** Foundation and appearing in the file LICENSE.GPL2 and LICENSE.GPL3
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-2.0.html and
-** https://www.gnu.org/licenses/gpl-3.0.html.
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
+// Copyright (C) 2016 The Qt Company Ltd.
+// SPDX-License-Identifier: LicenseRef-Qt-Commercial OR LGPL-3.0-only OR GPL-2.0-only OR GPL-3.0-only
 
 #include "qglobal.h"
 
@@ -45,6 +9,7 @@
 #include "qgraphicslayoutitem_p.h"
 #include "qwidget.h"
 #include "qgraphicswidget.h"
+#include "qgraphicsitem_p.h"
 
 #include <QtDebug>
 
@@ -105,7 +70,7 @@ static void normalizeHints(qreal &minimum, qreal &preferred, qreal &maximum, qre
     \internal
 */
 QGraphicsLayoutItemPrivate::QGraphicsLayoutItemPrivate(QGraphicsLayoutItem *par, bool layout)
-    : parent(par), userSizeHints(0), isLayout(layout), ownedByLayout(false), graphicsItem(0)
+    : parent(par), userSizeHints(nullptr), isLayout(layout), ownedByLayout(false), graphicsItem(nullptr)
 {
 }
 
@@ -193,7 +158,7 @@ QSizeF *QGraphicsLayoutItemPrivate::effectiveSizeHints(const QSizeF &constraint)
 /*!
     \internal
 
-    Returns the parent item of this layout, or 0 if this layout is
+    Returns the parent item of this layout, or \nullptr if this layout is
     not installed on any widget.
 
     If this is the item that the layout is installed on, it will return "itself".
@@ -214,7 +179,7 @@ QGraphicsItem *QGraphicsLayoutItemPrivate::parentItem() const
     while (parent && parent->isLayout()) {
         parent = parent->parentLayoutItem();
     }
-    return parent ? parent->graphicsItem() : 0;
+    return parent ? parent->graphicsItem() : nullptr;
 }
 
 /*!
@@ -368,7 +333,7 @@ bool QGraphicsLayoutItemPrivate::hasWidthForHeight() const
     passing a QGraphicsLayoutItem pointer to QGraphicsLayoutItem's
     protected constructor, or by calling setParentLayoutItem(). The
     parentLayoutItem() function returns a pointer to the item's layoutItem
-    parent. If the item's parent is 0 or if the parent does not inherit
+    parent. If the item's parent is \nullptr or if the parent does not inherit
     from QGraphicsItem, the parentLayoutItem() function then returns \nullptr.
     isLayout() returns \c true if the QGraphicsLayoutItem subclass is itself a
     layout, or false otherwise.
@@ -737,7 +702,7 @@ QRectF QGraphicsLayoutItem::geometry() const
     This virtual function provides the \a left, \a top, \a right and \a bottom
     contents margins for this QGraphicsLayoutItem. The default implementation
     assumes all contents margins are 0. The parameters point to values stored
-    in qreals. If any of the pointers is 0, that value will not be updated.
+    in qreals. If any of the pointers is \nullptr, that value will not be updated.
 
     \sa QGraphicsWidget::setContentsMargins()
 */
@@ -826,8 +791,28 @@ void QGraphicsLayoutItem::updateGeometry()
 }
 
 /*!
-    Returns the parent of this QGraphicsLayoutItem, or 0 if there is no parent,
-    or if the parent does not inherit from QGraphicsLayoutItem
+    \since 6.0
+
+    Returns \c true if this item is empty, i.e whether it has no content and
+    should not occupy any space.
+
+    The default implementation returns \c true true if the item has been hidden,
+    unless its \l{sizePolicy()}{size policy} has retainSizeWhenHidden set to \c true
+
+    \sa sizePolicy()
+*/
+bool QGraphicsLayoutItem::isEmpty() const
+{
+    bool isHidden = false;
+    if (QGraphicsItem *item = graphicsItem())
+        isHidden = QGraphicsItemPrivate::get(item)->explicitlyHidden;
+
+    return isHidden && !sizePolicy().retainSizeWhenHidden();
+}
+
+/*!
+    Returns the parent of this QGraphicsLayoutItem, or \nullptr if there is
+    no parent, or if the parent does not inherit from QGraphicsLayoutItem
     (QGraphicsLayoutItem is often used through multiple inheritance with
     QObject-derived classes).
 
@@ -917,7 +902,7 @@ QGraphicsItem *QGraphicsLayoutItem::graphicsItem() const
  * advantage of the automatic reparenting capabilities of QGraphicsLayout it
  * should set this value.
  * Note that if you delete \a item and not delete the layout item, you are
- * responsible of calling setGraphicsItem(0) in order to avoid having a
+ * responsible of calling setGraphicsItem(\nullptr) in order to avoid having a
  * dangling pointer.
  *
  * \sa graphicsItem()