Looks good to me. Ship it! This was part of some of my defunct dispatcher patches at one point. I think this is a lot cleaner than casting all the time anyway. --Jason <div class="gmail_quote">On Mar 28, 2013 8:58 AM, "Giulio Camuffo" <<a href="mailto:giuliocamuffo@gmail.com">giuliocamuffo@gmail.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> This is needed to make C++ programs that include wayland-server.h build: C++ does not allow conversions from data pointers to function pointers. --- This is the cleanest solution, imho. But if there are problems related to Wayland internals, or libffi, or something else, there are two other options: use a union { void *obj; void (* const *func)(); }; in wl_resource_init() in wayland-server.h to convert between the two (really ugly imho) or uninline wl_resource_init() and put the implementation somewhere the c++ compiler doesn't see. src/wayland-util.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/wayland-util.h b/src/wayland-util.h index 257a5d1..4728aaa 100644 --- a/src/wayland-util.h +++ b/src/wayland-util.h @@ -56,7 +56,7 @@ struct wl_interface { struct wl_object { const struct wl_interface *interface; - void (* const * implementation)(void); + const void *implementation; uint32_t id; }; -- 1.8.2 _______________________________________________ wayland-devel mailing list <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a> </blockquote></div>