On Sun, 15 Jan 2017 14:52:40 +0100 Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote:
> On 21/12/2016 15:40, Pekka Paalanen wrote: > > From: Pekka Paalanen <pekka.paala...@collabora.co.uk> > > > > Move the calls to set_title() and set_pid() out of > > weston_wm_window_read_properties() and into the three callers, each > > slightly different. > > > > xserver_map_shell_surface(): already calls these functions after > > creating the shell surface, so need to add calls. > > typo: “so need” → “so no need” Oh yeah! > > weston_wm_handle_map_request(): can be called only on unmapped (in X11) > > Windows, so no need to add calls. > > > > weston_wm_window_draw_decoration(): window->shsurf and window->surface > > are either both set or both NULL, so the check for window->shsurf is > > removed when moving the set_title() and set_pid() calls under a > > window->surface check. > > > > Signed-off-by: Pekka Paalanen <pekka.paala...@collabora.co.uk> > > Nice cleanup: > Reviewed-by: Quentin Glidic <sardemff7+...@sardemff7.net> Thank you, pq
pgppbq6bDtfx5.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel