Hi,

> It will now update the screen info, then detect and update output
> devices just like it does in CompScreen::init. This allows compiz to
> do the right thing when an external monitor or projector is plugged in
> and enabled using the 'gnome-display-properties' dialog or the
> 'xrandr' command line tool.
>
> Call XRRSelectInput in CompScreen::init, asking for
> RRScreenChangeNotify events. Without this those events are never given
> to compiz. That means that the switch case for it in
> plugins/composite/screen.cpp was never triggered, until now.

What is the exact problem this patch is trying to solve? I never had a 
problem with dynamically adding/removing monitors, because compiz is 
listening for ConfigureNotify on the root window, and some esoterical 
corner cases aside, the root window will change its size anyway when 
hotplugging monitors.

I don't see a problem in adding that patch (it's obviously correct), I 
just want to understand the reasoning.

Regards,

Danny
_______________________________________________
dev mailing list
[email protected]
http://lists.compiz.org/mailman/listinfo/dev

Reply via email to