Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-22 Thread Bastien Nocera
On Wed, 2007-11-21 at 21:29 -0500, Matthias Clasen wrote: snip Pascal's patch just wasn't very far along...he said so himself when he attached it. I reviewed it and he never came back to it. I waited until SoC was over to not step on his toes, and then reworked his patch myself. My reworked

Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-21 Thread Soeren Sandmann
Bastien Nocera [EMAIL PROTECTED] writes: On Wed, 2007-11-21 at 00:39 +0100, Soeren Sandmann wrote: Hi I have been doing some work on adding RandR 1.2 support to GTK+. There is an initial patch here: http://www.gnome.org/~ssp/randr/gtk.patch Was that based on the patch in

Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-21 Thread Bastien Nocera
On Wed, 2007-11-21 at 21:15 +0100, Soeren Sandmann wrote: Bastien Nocera [EMAIL PROTECTED] writes: On Wed, 2007-11-21 at 00:39 +0100, Soeren Sandmann wrote: Hi I have been doing some work on adding RandR 1.2 support to GTK+. There is an initial patch here:

Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-21 Thread Matthias Clasen
On Nov 21, 2007 7:46 PM, Bastien Nocera [EMAIL PROTECTED] wrote: It's the same API, but the code is different. I'll post it to bugzilla. Mind explaining what's different? Did you use the code in the bug as a starting point, or is it completely new code? If it's all new code, what was so

Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-20 Thread Soeren Sandmann
Hi I have been doing some work on adding RandR 1.2 support to GTK+. There is an initial patch here: http://www.gnome.org/~ssp/randr/gtk.patch The interesting part is perhaps the API: A new signal on GdkScreen: void (*monitors_changed) (GdkScreen *screen); that is emitted whenever

Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-20 Thread Bastien Nocera
On Wed, 2007-11-21 at 00:39 +0100, Soeren Sandmann wrote: Hi I have been doing some work on adding RandR 1.2 support to GTK+. There is an initial patch here: http://www.gnome.org/~ssp/randr/gtk.patch Was that based on the patch in the bugzilla:

Re: Monitor and projector hotplug (aka RandR 1.2 support for GTK+)

2007-11-20 Thread Matthias Clasen
On 21 Nov 2007 00:39:29 +0100, Soeren Sandmann [EMAIL PROTECTED] wrote: Comments appreciated. Looks pretty close to what Pascal and I had worked out earlier in bugzilla, so I'd say it looks fine api wise. The only things that jumped out in the patch were the debug printfs that are still in