At 1238854563 time_t, Maarten Maathuis wrote:
> @@ -105,20 +105,20 @@ wibox_setposition(wibox_t *wibox, position_t p)
>   * \param phys_screen Physical screen number.
>   */
>  static void
> -wibox_systray_kickout(int phys_screen)
> +wibox_systray_kickout(int screen, int phys_screen)

What the point of phys_screen if you do not use it anymore?

Anyway this patch is bad.
We handle only a systray by phys_screen, and in Xinerama mode we have only
one phys_screen.
There's no reason to change that, and if you go in this direction, you
should also give a look at systray_init(), which in Xinerama mode only
init one systray parent window.

-- 
Julien Danjou
// ᐰ <jul...@danjou.info>   http://julien.danjou.info
// 9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD
// When I get sad, I stop being sad and be awesome instead. True story.

Attachment: signature.asc
Description: Digital signature

Reply via email to