Lists

Terms and Conditions
Lists hosted on this site
Email the Postmaster
Tips for posting to public mailing lists

Re: XP_EVENT_WINDOW_MOVED

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: XP_EVENT_WINDOW_MOVED

Subject: Re: XP_EVENT_WINDOW_MOVED
From: Ben Byer <email@hidden>
Date: Thu, 22 Nov 2007 02:37:10 -0800


On Nov 21, 2007, at 10:26 PM, Jeremy Huddleston wrote:

Hi Ben,
You made a change to XP_EVENT_WINDOW_MOVED handling in xprScreen.c to help support spaces. This was in the following commit: http://gitweb.freedesktop.org/?p=xorg/xserver.git;a=commit;h=2d50ea8013e7c1639d570e227b53b037fb567565
I noticed it reverted in this commit:
http://gitweb.freedesktop.org/?p=xorg/xserver.git;a=commit;h=dc48ad4a45d37a7fc65a3a0455fccd4aca09962f
The comments for the second commit were 'fix compiler warnings'. Was that revert accidental, or was there some other reason you reverted the change? I'm going through all our branch commits to make sure master is up to date with our branch fixes, and I wasn't sure about this one.

Yeah, that was deliberate. My first attempt at handling XP_EVENT_WINDOW_MOVED required that we pass the new (X,Y) coords as parameters; when I ripped that code out and replaced it with the code from the old X11.app, that code did not need those extra parameters. -- Ben Byer CoreOS / BSD Technology Group, XDarwin maintainer

_______________________________________________
Do not post admin requests to the list. They will be ignored.
X11-users mailing list      (email@hidden)
This email sent to email@hidden



Prev by Date:
master branch

Next by Date:
Re: Keys and remapping them (was Re: Daydreaming)

Previous by thread:
Re: master branch

Next by thread:
Calc in full screen

Index(es):

Date
Thread