isa/mutter
1
0
Fork 0
mirror of https://github.com/brl/mutter.git synced 2024-11-27 02:20:43 -05:00
Code Issues Packages Projects Releases Wiki Activity

Be more tolerant about natural_width < min_width

Browse Source 
Due to the accumulation of floating point errors, natural_width
and min_width can diverge significantly even if the math for
computing them is correct. So just clamp natural_width to
min_width instead of warning about it.

http://bugzilla.openedhand.com/show_bug.cgi?id=1632

Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>
This commit is contained in:
Owen W. Taylor 2009-06-06 11:37:18 -04:00 committed by Emmanuele Bassi
parent 583a86b537
commit 2b7e98f4b3
1 changed files with 10 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
clutter/clutter-actor.c
View File

@ -4464,8 +4464,6 @@ clutter_actor_get_preferred_size (ClutterActor *self,
*natural_height_p = natural_height; *natural_height_p = natural_height;
} }
#define FLOAT_EPSILON (1e-4) /* 1/1000 pixel is the precision used */
/** /**
* clutter_actor_get_preferred_width: * clutter_actor_get_preferred_width:
* @self: A #ClutterActor * @self: A #ClutterActor
@ -4513,14 +4511,11 @@ clutter_actor_get_preferred_width (ClutterActor *self,
&min_width, &min_width,
&natural_width); &natural_width);
if (natural_width < min_width - FLOAT_EPSILON) /* Due to accumulated float errors, it's better not to warn
{ * on this, but just fix it.
g_warning ("Actor of type %s reported a natural width " */
"of %.2f px lower than min width %.2f px", if (natural_width < min_width)
G_OBJECT_TYPE_NAME (self), natural_width = min_width;
natural_width,
min_width);
}
if (!priv->min_width_set) if (!priv->min_width_set)
priv->request_min_width = min_width; priv->request_min_width = min_width;
@ -4585,14 +4580,11 @@ clutter_actor_get_preferred_height (ClutterActor *self,
&min_height, &min_height,
&natural_height); &natural_height);
if (natural_height < min_height - FLOAT_EPSILON) /* Due to accumulated float errors, it's better not to warn
{ * on this, but just fix it.
g_warning ("Actor of type %s reported a natural height " */
"of %.2f px lower than min height %.2f px", if (natural_height < min_height)
G_OBJECT_TYPE_NAME (self), natural_height = min_height;
natural_height,
min_height);
}
if (!priv->min_height_set) if (!priv->min_height_set)
priv->request_min_height = min_height; priv->request_min_height = min_height;