On 17 February 2016 at 14:04, Keith Packard <[email protected]> wrote: > This lets clients know that the layout of the monitors on the screen > has changed so they can adapt appropriately. >
Seems like a good thing to do. Reviewed-by: Dave Airlie <[email protected]> > Signed-off-by: Keith Packard <[email protected]> > --- > randr/rrmonitor.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/randr/rrmonitor.c b/randr/rrmonitor.c > index c37dcf8..f8743d0 100644 > --- a/randr/rrmonitor.c > +++ b/randr/rrmonitor.c > @@ -721,7 +721,9 @@ ProcRRSetMonitor(ClientPtr client) > monitor->geometry.mmHeight = stuff->monitor.heightInMillimeters; > > r = RRMonitorAdd(client, screen, monitor); > - if (r != Success) > + if (r == Success) > + RRSendConfigNotify(screen); > + else > RRMonitorFree(monitor); > return r; > } > @@ -745,5 +747,8 @@ ProcRRDeleteMonitor(ClientPtr client) > return BadAtom; > } > > - return RRMonitorDelete(client, screen, stuff->name); > + r = RRMonitorDelete(client, screen, stuff->name); > + if (r == Success) > + RRSendConfigNotify(screen); > + return r; > } > -- > 2.7.0 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: https://lists.x.org/mailman/listinfo/xorg-devel _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
