[PATCH 13/15] worker: move shadow_new() and container_new()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>

---
 server/red_worker.c | 61 ++++++++++++-----------------------------------------
 server/tree.c       | 39 ++++++++++++++++++++++++++++++++++
 server/tree.h       |  9 ++++++++
 3 files changed, 61 insertions(+), 48 deletions(-)

diff --git a/server/red_worker.c b/server/red_worker.c
index f40cd30..e9ef821 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -1654,24 +1654,6 @@ static void exclude_region(RedWorker *worker, Ring *ring, RingItem *ring_item, Q
     }
 }
 
-static inline Container *__new_container(RedWorker *worker, DrawItem *item)
-{
-    Container *container = spice_new(Container, 1);
-    worker->containers_count++;
-    container->base.type = TREE_ITEM_TYPE_CONTAINER;
-    container->base.container = item->base.container;
-    item->base.container = container;
-    item->container_root = TRUE;
-    region_clone(&container->base.rgn, &item->base.rgn);
-    ring_item_init(&container->base.siblings_link);
-    ring_add_after(&container->base.siblings_link, &item->base.siblings_link);
-    ring_remove(&item->base.siblings_link);
-    ring_init(&container->items);
-    ring_add(&container->items, &item->base.siblings_link);
-
-    return container;
-}
-
 static inline int is_opaque_item(TreeItem *item)
 {
     return item->type == TREE_ITEM_TYPE_CONTAINER ||
@@ -2979,8 +2961,8 @@ static inline int red_current_add(RedWorker *worker, Ring *ring, Drawable *drawa
                     continue;
                 }
                 spice_assert(IS_DRAW_ITEM(sibling));
-                if (!((DrawItem *)sibling)->container_root) {
-                    container = __new_container(worker, (DrawItem *)sibling);
+                if (!DRAW_ITEM(sibling)->container_root) {
+                    container = container_new(DRAW_ITEM(sibling));
                     if (!container) {
                         spice_warning("create new container failed");
                         region_destroy(&exclude_rgn);
@@ -3014,7 +2996,7 @@ static inline int red_current_add(RedWorker *worker, Ring *ring, Drawable *drawa
          * before calling red_detach_streams_behind
          */
         __current_add_drawable(worker, drawable, ring);
-        if (drawable->surface_id == 0) {
+        if (is_primary_surface(worker, drawable->surface_id)) {
             red_detach_streams_behind(worker, &drawable->tree_item.base.rgn, drawable);
         }
     }
@@ -3032,31 +3014,6 @@ static void add_clip_rects(QRegion *rgn, SpiceClipRects *data)
     }
 }
 
-static inline Shadow *__new_shadow(RedWorker *worker, Drawable *item)
-{
-    RedDrawable *red_drawable = item->red_drawable;
-    SpicePoint delta = {
-        .x = red_drawable->u.copy_bits.src_pos.x - red_drawable->bbox.left,
-        .y = red_drawable->u.copy_bits.src_pos.y - red_drawable->bbox.top
-    };
-
-    if (!delta.x && !delta.y) {
-        return NULL;
-    }
-
-    Shadow *shadow = spice_new(Shadow, 1);
-    worker->shadows_count++;
-    shadow->base.type = TREE_ITEM_TYPE_SHADOW;
-    shadow->base.container = NULL;
-    shadow->owner = &item->tree_item;
-    region_clone(&shadow->base.rgn, &item->tree_item.base.rgn);
-    region_offset(&shadow->base.rgn, delta.x, delta.y);
-    ring_item_init(&shadow->base.siblings_link);
-    region_init(&shadow->on_hold);
-    item->tree_item.shadow = shadow;
-    return shadow;
-}
-
 static inline int red_current_add_with_shadow(RedWorker *worker, Ring *ring, Drawable *item)
 {
 #ifdef RED_WORKER_STAT
@@ -3064,11 +3021,18 @@ static inline int red_current_add_with_shadow(RedWorker *worker, Ring *ring, Dra
     ++worker->add_with_shadow_count;
 #endif
 
-    Shadow *shadow = __new_shadow(worker, item);
+    RedDrawable *red_drawable = item->red_drawable;
+    SpicePoint delta = {
+        .x = red_drawable->u.copy_bits.src_pos.x - red_drawable->bbox.left,
+        .y = red_drawable->u.copy_bits.src_pos.y - red_drawable->bbox.top
+    };
+
+    Shadow *shadow = shadow_new(&item->tree_item, &delta);
     if (!shadow) {
         stat_add(&worker->add_stat, start_time);
         return FALSE;
     }
+    worker->shadows_count++;
     // item and his shadow must initially be placed in the same container.
     // for now putting them on root.
 
@@ -3076,6 +3040,7 @@ static inline int red_current_add_with_shadow(RedWorker *worker, Ring *ring, Dra
     if (is_primary_surface(worker, item->surface_id)) {
         red_detach_streams_behind(worker, &shadow->base.rgn, NULL);
     }
+
     ring_add(ring, &shadow->base.siblings_link);
     __current_add_drawable(worker, item, ring);
     if (item->tree_item.effect == QXL_EFFECT_OPAQUE) {
@@ -3085,7 +3050,7 @@ static inline int red_current_add_with_shadow(RedWorker *worker, Ring *ring, Dra
         region_destroy(&exclude_rgn);
         red_streams_update_visible_region(worker, item);
     } else {
-        if (item->surface_id == 0) {
+        if (is_primary_surface(worker, item->surface_id)) {
             red_detach_streams_behind(worker, &item->tree_item.base.rgn, item);
         }
     }
diff --git a/server/tree.c b/server/tree.c
index ed7d39a..64fa6db 100644
--- a/server/tree.c
+++ b/server/tree.c
@@ -180,3 +180,42 @@ void tree_item_dump(TreeItem *item)
     spice_return_if_fail(item != NULL);
     tree_foreach(item, dump_item, &di);
 }
+
+Shadow* shadow_new(DrawItem *item, const SpicePoint *delta)
+{
+    spice_return_val_if_fail(item->shadow == NULL, NULL);
+    if (!delta->x && !delta->y) {
+        return NULL;
+    }
+
+    Shadow *shadow = spice_new(Shadow, 1);
+
+    shadow->base.type = TREE_ITEM_TYPE_SHADOW;
+    shadow->base.container = NULL;
+    shadow->owner = item;
+    region_clone(&shadow->base.rgn, &item->base.rgn);
+    region_offset(&shadow->base.rgn, delta->x, delta->y);
+    ring_item_init(&shadow->base.siblings_link);
+    region_init(&shadow->on_hold);
+    item->shadow = shadow;
+
+    return shadow;
+}
+
+Container* container_new(DrawItem *item)
+{
+    Container *container = spice_new(Container, 1);
+
+    container->base.type = TREE_ITEM_TYPE_CONTAINER;
+    container->base.container = item->base.container;
+    item->base.container = container;
+    item->container_root = TRUE;
+    region_clone(&container->base.rgn, &item->base.rgn);
+    ring_item_init(&container->base.siblings_link);
+    ring_add_after(&container->base.siblings_link, &item->base.siblings_link);
+    ring_remove(&item->base.siblings_link);
+    ring_init(&container->items);
+    ring_add(&container->items, &item->base.siblings_link);
+
+    return container;
+}
diff --git a/server/tree.h b/server/tree.h
index e10fa1c..9ee6007 100644
--- a/server/tree.h
+++ b/server/tree.h
@@ -52,11 +52,17 @@ struct Shadow {
     DrawItem* owner;
 };
 
+#define IS_SHADOW(item) ((item)->type == TREE_ITEM_TYPE_SHADOW)
+#define SHADOW(item) ((Container*)(item))
+
 struct Container {
     TreeItem base;
     Ring items;
 };
 
+#define IS_CONTAINER(item) ((item)->type == TREE_ITEM_TYPE_CONTAINER)
+#define CONTAINER(item) ((Container*)(item))
+
 struct DrawItem {
     TreeItem base;
     uint8_t effect;
@@ -65,6 +71,7 @@ struct DrawItem {
 };
 
 #define IS_DRAW_ITEM(item) ((item)->type == TREE_ITEM_TYPE_DRAWABLE)
+#define DRAW_ITEM(item) ((DrawItem*)(item))
 
 typedef struct DependItem {
     Drawable *drawable;
@@ -101,5 +108,7 @@ struct Drawable {
 };
 
 void       tree_item_dump                           (TreeItem *item);
+Shadow*    shadow_new                               (DrawItem *item, const SpicePoint *delta);
+Container* container_new                            (DrawItem *item);
 
 #endif /* TREE_H_ */
-- 
2.4.3

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]