On Sun, 2014-08-03 at 10:39 +0300, Pekka Paalanen wrote: > On Mon, 21 Jul 2014 13:23:23 -0700 > Jason Ekstrand <ja...@jlekstrand.net> wrote: > > > I think the way it's handled in subsurfaces allows you to create a > > subsurface from a surface, delete the surface, and then re-use the surface > > as something else. If I recall correctly, weston allows a fair amount of > > surface re-use right now. > > > > That said, I'm not opposed to not allowing it for xdg surfaces, but I do > > think they should still have an explicit destructor. > > Like I wrote in my recent blog post [1], not completely unrelated to > this discussion in irc, IIRC, I strongly suggest adding destructor > protocol unless you can prove that having destructor protocol is > actively harmful.
Thanks for that post, as always quite enlightening :) > As for wl_surface re-use after destroying a role, I have found that > resetting a role to allow re-use is much easier and cleaner than > not to, in Weston code. That's why I've been silently advocating to > allow re-use, but at the same I don't really expect clients to use > that. Yes, conflicting... so I suppose I don't have a real strong > preference there. Removing a role loses all role-specific > information, so whether the wl_surface get re-used or re-created > does not make any difference. So with your blogpost you've nicely managed to convince me that we should keep the destroy request regardless of the semantics of it.. However, whether or not re-use is allowed should probably be made quite explicit and especially in the case of weston enforced (if not). I'm generally quite wary about having e.g. Weston be more permissive then other compositors (e.g. mutter) as that is likely to just lead to incompatibilities. If you continue that line of reasoning, it's probably best to not allow it simply such that the odd clients that do use re-use don't hit barely tested codepaths on compositors. -- Sjoerd Simons <sjoerd.sim...@collabora.co.uk> Collabora Ltd.
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel
