[PATCH v2 xinput] man: document XI2 options

Peter Hutterer peter.hutterer at who-t.net
Thu Feb 4 14:42:27 PST 2010


On Wed, Feb 03, 2010 at 11:59:40PM -0800, Jeremy Huddleston wrote:
> 
> On Feb 3, 2010, at 21:06, Peter Hutterer wrote:
> 
> > +.B --reattach \fIdevice\fP \fImaster\fP
> > +Reattach the given device to the given master device.
> 
> Formatting throughout this file seems inconsistent with respect to hilighting arguments in the descriptions.  Shouldn't this be more like:
> 
> .B --reattach \fIdevice\fP \fImaster\fP
> Reattach \fIdevice\fP to the \fImaster\fP device.
> 
> which better matches what's already in the man page:
> .B --get-feedbacks \fIdevice\fP
> Display the feedbacks of \fIdevice\fP.
> 
> I just picked this one line as an example, but there are inconsistencies like this throughout.  As example, this is currently in the man page:

you're right, was a bit slack. I went through and fixed up the highlighting
and a bit of the wording. The man page now refers to "master" and "slave"
where needed instead of just "device". Fixed patch is below.  For the other
problematic highlighting (mostly in the property calls) I'd rather have a
separate patch go in though.



More information about the xorg-devel mailing list