Xfwm4 compositor zooming patch

[Alistair Buxton]

All that stuff now fixed on zoom3 branch:

https://github.com/ali1234/xfwm4/tree/zoom3

On 25 November 2013 18:39, Alistair Buxton <a.j.buxton at gmail.com> wrote:
> On 25 November 2013 09:03, Nick Schermer <nick at xfce.org> wrote:
>> On Sat, Nov 16, 2013 at 5:56 PM, Alistair Buxton <a.j.buxton at gmail.com>
>> wrote:
>>>
>>> I've made a new branch with some fixes:
>>> https://github.com/ali1234/xfwm4/commits/zoom2
>>>
>>> I think this still needs more testing before being merged, as the one
>>> person who reported it doesn't work got a completely black screen
>>> which is a pretty bad regression.
>>
>>
>> But the regression only occurs during zooming, so its not that bad. The
>> problem with the branch is that not a lot of people will test it.
>
> No, they reported regression happens all the time as soon as xfwm4
> starts. This doesn't make much sense, but there you go. I agree nobody
> will test it :(
>
>
>>> Xfwm4 compositor already gets the refresh rate of the display using
>>> randr. It is probably better to use that where available, rather than
>>> a hardcoded value.
>>>
>>
>> Maybe the full refresh rate is a bit too much, half of it feels responsive
>> enough here. Also to cut the calls to XQueryPointer.
>
> Ok. There's actually a bug with this little bit of code: It's possible
> to have RANDR but not LIBDRM, which causes the code to not compile.
> It's just a matter of moving around some checks. I'll fix this and
> make a new branch.
>
> --
> Alistair Buxton
> a.j.buxton at gmail.com



-- 
Alistair Buxton
a.j.buxton at gmail.com