discomfitor pushed a commit to branch master.
Author: Mike Blumenkrantz <zm...@osg.samsung.com>
Date: Tue Jan 19 12:11:27 2016 -0500
Revert "Fix issue of compositor surface create passing wrong parameter to"
This reverts commit 67170f40a1a901101591c8102631dbec8e49794b.
this was changed intentionally to use the resource pointer in order to fix
issue where external clients would reuse the same surface id,
thereby breaking the compositor with duplicate entries in the pixmap hash.
note, however, that internal windows in wayland DO use an int type pixmap
this is easily detected by checking the pid of the client for a window
checks. this is necessary in order to be able to flag internal clients as
while still being able to match them with their surface id
also, uintptr_t is NOT indicative of an int type being used, it's an
int type which has the same size as a pointer, allowing casts between ints
src/bin/e_comp_wl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/bin/e_comp_wl.c b/src/bin/e_comp_wl.c
index c282ef8..579c4ff 100644
@@ -1480,7 +1480,7 @@ _e_comp_wl_compositor_cb_surface_create(struct wl_client
*client, struct wl_reso
/* try to create new pixmap */
- if (!(ep = e_pixmap_new(E_PIXMAP_TYPE_WL, (uintptr_t)id)))
+ if (!(ep = e_pixmap_new(E_PIXMAP_TYPE_WL, res)))
ERR("Could not create new pixmap");