[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mandoc-tech
Subject:    Re: [PATCH] .Sm spacing
From:       Kristaps Dzonsons <kristaps () bsd ! lv>
Date:       2010-07-13 23:13:31
Message-ID: 4C3CF31B.6090403 () bsd ! lv
[Download RAW message or body]

> here is a very small patch to fix .Sm spacing.
> 
> When saying ".Sm on", you expect spacing to start right away, not after
> the following token.  Thus, we need to clear the TERMP_NOSPACE flag in
> addition to clearing TERMP_NONOSPACE.  On the other hand, we still do
> not want spacing at the beginning of a line (in the sense of "line" as
> defined in termp_flushln), so leave TERMP_NOSPACE intact when in
> column 0.
> 
> In OpenBSD, this fixes the following pages for me:
> 
>  - chmod(1):
>    The display right below "Symbolic modes",
>     - chmod <[who]op[perm]>,[[who]op[perm]],[...]file [...]
>     + chmod <[who]op[perm]>,[[who]op[perm]],[...] file [...]
> 
>  - ksh(1):
>    A couple of cases where text followed stuff like
>    ${name[expr]}, ${name=value}, <var><op>=<expr>
>    without a space.
> 
>  - tar(1):
>    The first SYNOPSIS:
>     - tar {crtux}[014578befHhjLmOoPpqsvwXZz][blocking-factor
>     -     | archive | replstr] [-C directory] [-I file] [file ...]
>     + tar {crtux}[014578befHhjLmOoPpqsvwXZz]
>     +     [blocking-factor | archive | replstr] [-C directory] [-I file]
>     +     [file ...]
> 
>  - ps(1):
>    The SYNOPSIS:
>     - ps [-aCcehjkLlmrSTuvwx][-M core] [-N system] [-O fmt] [-o fmt] [-p pid]
>     + ps [-aCcehjkLlmrSTuvwx] [-M core] [-N system] [-O fmt] [-o fmt] [-p pid]
>          [-t tty] [-U username] [-W swap]

Patch looks fine by me.  But "ok kristaps" only on mdoc.7 addition :-)
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv

[prev in list] [next in list] [prev in thread] [next in thread]