On 10/ 7/11 05:41 AM, Michal Suchanek wrote:
On 6 October 2011 23:39, Alan Coopersmith<[email protected]>  wrote:
On 10/ 6/11 07:54 AM, Michal Suchanek wrote:

this patch documents the option introduced in commit 8976e97.

+++ b/hw/xfree86/man/Xorg.man

Since the option is in the core server argument processing (os/utils.c,
not under hw/xfree86/common), it should probably go in man/Xserver.man
which is referenced for common args from the
Xorg/Xwin/Xvfb/Xnest/Xephyr/etc.
man pages.

This updated patch puts the option in the common man page.

Thanks.   If you don't mind a little grammar nitpicking, a couple suggestions:

+.BI \-background " none"
+If the driver supports it it will be asked to not clear the background on X 
startup.

Having "it" twice in a row, referring to different things is a bit awkward.
Perhaps:

  Asks the driver not to clear the background on startup, if the driver supports
  that.

+May be useful for smooth transition with eg. fbdev driver.
+For security reasons this is not the default. The screen content might show 
previous user session.

I'd change that to:
  For security reasons this is not the default, as the screen contents
  might show a previous user session.

--
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to