From 7e7d093e1f371bc44382e1e74795de4ff750bec7 Mon Sep 17 00:00:00 2001 From: Nicola Fontana Date: Sat, 9 Jan 2010 12:00:09 +0100 Subject: [PATCH] [AdgStroke] Stylish improvements Moved special case on its own instead of polluting the normal flow inside the local_changed() callback. --- adg/adg-stroke.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/adg/adg-stroke.c b/adg/adg-stroke.c index a3d526f7..4ce4a00f 100644 --- a/adg/adg-stroke.c +++ b/adg/adg-stroke.c @@ -286,10 +286,8 @@ local_changed(AdgEntity *entity) /* For simple translation, avoid the entity invalidation: * translate the extents of the same vector instead */ - if (old.xx != new->xx || old.yy != new->yy || - old.xy != new->xy || old.yx != new->yx) { - adg_entity_invalidate(entity); - } else { + if (old.xx == new->xx && old.yy == new->yy && + old.xy == new->xy && old.yx == new->yx) { CpmlExtents extents; cpml_extents_copy(&extents, adg_entity_extents(entity)); @@ -299,6 +297,8 @@ local_changed(AdgEntity *entity) adg_entity_set_extents(entity, &extents); } + + adg_entity_invalidate(entity); } static void -- 2.11.4.GIT