[Xfce4-commits] [apps/xfdashboard] 01/02: Do not set allocation on container within the layout manager's allocation function at XfdashboardDynamicTableLayout.

[noreply at xfce.org]

This is an automated email from the git hooks/post-receive script.

nomad pushed a commit to branch master
in repository apps/xfdashboard.

commit 2929b13715f4cf003bb311e48f230d684d537ca4
Author: Stephan Haller <nomad at froevel.de>
Date:   Mon Feb 15 09:10:50 2016 +0100

    Do not set allocation on container within the layout manager's allocation function at XfdashboardDynamicTableLayout.
    
    That is a nasty bug as it will call the allocation function recursive in an infinite loop because setting the allocation on the container will call the layout manager's allocation function again and again and again ...
---
 xfdashboard/dynamic-table-layout.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/xfdashboard/dynamic-table-layout.c b/xfdashboard/dynamic-table-layout.c
index 55716e0..66e66e0 100644
--- a/xfdashboard/dynamic-table-layout.c
+++ b/xfdashboard/dynamic-table-layout.c
@@ -355,13 +355,6 @@ static void _xfdashboard_dynamic_table_layout_allocate(ClutterLayoutManager *sel
 															width,
 															height);
 
-	/* Set container allocation */
-	childAllocation.x1=0;
-	childAllocation.y1=0;
-	childAllocation.x2=g_array_index(priv->columnCoords, gfloat, priv->columns)-priv->columnSpacing;
-	childAllocation.y2=g_array_index(priv->rowCoords, gfloat, priv->rows)-priv->rowSpacing;
-	clutter_actor_allocate(CLUTTER_ACTOR(inContainer), &childAllocation, inFlags);
-
 	/* Determine allocation for each visible child */
 	i=0;
 	clutter_actor_iter_init(&iter, CLUTTER_ACTOR(inContainer));

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.