QGraphicsAnchorLayout: size hint updated in a simplified graph

This commit makes setAnchorSizeHintsFromItems() support a simplified
graph. Instead of going recursively, since AnchorData doesn't have all
needed information, we make a list with all anchors/vertexpairs with
dependencies first. That way we are sure to fill all information, and
the group anchors can peek at its children anchors (dependencies) and
they'll have updated information.

This solutions doesn't require refreshSizeHints(). Alternative
solutions that we might consider are: complementing the AnchorData
information or making refreshSizeHints() take two AnchorVertex as
parameters and use the recursive approach.

Signed-off-by: Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
Reviewed-by: Artur Duque de Souza <artur.souza@openbossa.org>

1 files changed, 0 insertions, 5 deletions

diff --git a/src/gui/graphicsview/qgraphicsanchorlayout_p.h b/src/gui/graphicsview/qgraphicsanchorlayout_p.h
index 21a4a3f..3951910 100644
--- a/src/gui/graphicsview/qgraphicsanchorlayout_p.h
+++ b/src/gui/graphicsview/qgraphicsanchorlayout_p.h
@@ -147,8 +147,6 @@ struct AnchorData : public QSimplexVariable {
 
     virtual void updateChildrenSizes() { };
 
-    virtual void refreshSizeHints() { };
-
     void dump(int indent = 2);
 
     inline QString toString() const;
@@ -199,7 +197,6 @@ struct SequentialAnchorData : public AnchorData
     }
 
     virtual void updateChildrenSizes();
-    virtual void refreshSizeHints();
 
     void setVertices(const QVector<AnchorVertex*> &vertices)
     {
@@ -223,8 +220,6 @@ struct ParallelAnchorData : public AnchorData
     }
 
     virtual void updateChildrenSizes();
-    virtual void refreshSizeHints();
-
 
     AnchorData* firstEdge;
     AnchorData* secondEdge;