Re: [Acecad-devel] [PATCH] Add options to inverse x & y axis, and swap them

[Frederic L. <fr...@st...>]

On Fri, 16 Sep 2005 09:52:35 +0000
Edouard TISSERANT <edo...@wa...> wrote:

> Merci Fred for the patch. What do you mean about AceCadPrivate structure ?
In xorg cvs, the device structure is named 'AceCadPrivate', whereas in Stef
work it is named 'AcecadPrivate'. Notice the 'c' of Acecad. 

> Your patch have also patched my mind about CVS organisation.
:) 
> I will commit xorg's acecad CVS version as HEAD initial commit for
> acecad-xinput module.
>
> Then I'll commit the old acecad-3.1.tgz as second version of HEAD.
> Differences between this two versions is carlo's A-series contrib and
> button maps.
> 
> Then I'll commit steph's 1.0.6 as third version of HEAD. Lot of changes
> IOGrabs, lot of rewrites, new config parameters, hotplug, auto-dev etc...
> 
> I wonder if it is still necessary to make a branch for Fred's changes.
> I'll test them before commit.
I think another branch isn't necessary as I can rewrite my work to fit
our cvs. I'll go by small step so it'll easily reversible and reviewable.

> Hotplug tools should be another module. Why not acecad-hotplug ?
Which hotplug tools ?
If I read correctly this page :
http://wiki.x.org/wiki/XInputHotplug
We need to fit deviceOn(), deviceOff() and others in deviceControl() to use
xadddev, and xrmdev.
If i understand correctly, XAddInputDevice() and others call deviceControl.
This is still alpha work in xorg.
Changing acecad driver to work with these is not a big change nor a big
worK.
Another cvs branch for hotplug is however good as device hotplug is not yet
well defined in xorg.

-- 
Frederic Leroy
Lost in France