From 86a31e34ba66c7e593cd93103365213c1c9d2d2f Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Sun, 12 Jul 2020 00:00:27 -0600 Subject: [PATCH] Canvas: fix recursive calling of Item::size_allocate() ConstraintPacker is the one Item-derivative that should NOT call Item::size_allocate_children() because it just did the size computation for its children and called their ::constrained() method, which in turns calls size_allocate() with the correct size --- libs/canvas/constraint_packer.cc | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libs/canvas/constraint_packer.cc b/libs/canvas/constraint_packer.cc index 814b5c3272..d310c883e7 100644 --- a/libs/canvas/constraint_packer.cc +++ b/libs/canvas/constraint_packer.cc @@ -307,7 +307,10 @@ ConstraintPacker::size_allocate (Rect const & r) PBD::Unwinder uw (in_alloc, true); double expanded_size; - Item::size_allocate (r); + if (_layout_sensitive) { + _position = Duple (r.x0, r.y0); + _allocation = r; + } if (!packed.empty()) {