On Sat, Feb 27, 2010 at 9:24 PM, Chris Bagwell <ch...@cnpbagwell.com> wrote:
> Ping, do you think the cleanup patch is acceptable then?

This was the question that I asked myself when I replied yesterday.
And I could not make the decision then.

> To summarize the patch: it was removing broken logic that looked at
> xorg.conf values for maxX/Y/Z because user values were always thrown
> away.

Yes, I understand the point. As you described in your commit comments,
the tablet-to-screen mapping use case can be covered by BottomX/Y
options.

> Based on responses so far and further reviews I've been doing, I think
> we can safely say there is no current need for user to be able to
> set/override those Max values reported by hardware... so no need to
> get the broken logic working.

So, reviewed-by: Ping Cheng <pingli...@gmail.com>.

Ping

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to