Re: [PATCH v2 1/3] xwayland: Fix the addition and removal of outputs

2015-05-24 Thread Dima Ryazanov
Some more context here: currently, the output_handle_done function is not really doing anything, but I haven't figured out yet how to fix it. These patches fix a few trivial bugs, but there's more work to be done. The call to RRScreenSizeNotify is a no-op: it expects the width/height of xwl_screen

Re: [PATCH v2 1/3] xwayland: Fix the addition and removal of outputs

2015-05-21 Thread Jasper St. Pierre
Expecting anything atomic from X11 in the first place is a wrong assumption. On Thu, May 21, 2015 at 6:18 AM, Marek Chalupa wrote: > Hi, > > On Sat, May 16, 2015 at 7:38 AM, Dima Ryazanov wrote: >> >> Add the output to the list when it's created rather than when its >> properties >> change (as p

Re: [PATCH v2 1/3] xwayland: Fix the addition and removal of outputs

2015-05-21 Thread Marek Chalupa
Hi, On Sat, May 16, 2015 at 7:38 AM, Dima Ryazanov wrote: > Add the output to the list when it's created rather than when its > properties > change (as pointed out by Marek Chalupa). > Remove the output from the list when it's destroyed. > > Signed-off-by: Dima Ryazanov > --- > hw/xwayland/xwa

[PATCH v2 1/3] xwayland: Fix the addition and removal of outputs

2015-05-15 Thread Dima Ryazanov
Add the output to the list when it's created rather than when its properties change (as pointed out by Marek Chalupa). Remove the output from the list when it's destroyed. Signed-off-by: Dima Ryazanov --- hw/xwayland/xwayland-output.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) d