mandoc/TODO - diff

Return to TODO CVS log

Up to [cvsweb.bsd.lv] / mandoc

Diff for /mandoc/TODO between version 1.89 and 1.176

-version 1.89, 2011/02/03 10:24:21
+version 1.176, 2014/08/09 14:24:53
 Line 4
 Line 4
 Line 4
  ************************************************************************
  ************************************************************************
- * parser bugs
+ * crashes
  ************************************************************************
- - It should be noted that in mandoc, the TBL macros are interpreted as
+ - The abort() in bufcat(), html.c, can be triggered via buffmt_includes()
-   real roff macros (e.g., `.  TS' is ok), while the troff preprocessor
+   by running -Thtml -Oincludes on a file containing a long .In argument.
-   only looks for `.TS' (static string).  This should be noted in the
+   Fixing this will probably require reworking the whole bufcat() concept.
-   COMPATIBILITY section.  The same applies for nascent `.EQ' (as it is
-   now, `.EN' is processed as a static string).
- - .TP before .SH is still FATAL in man(7)
-   reported by brad@  Sat, 15 Jan 2011 15:54:54 -0500
- - The mdoc(7) parser should handle `\.' as punctuation:
-     .Ql .if ${VAR} op something \.
-   should produce
-     `.if ${VAR} op something'.
-   not
-     `.if ${VAR} op something .'
-   as seen in make(1).
-   To escape the dot, one would use `\&.'.
- - the roff parser doesn't tolerate additional characters between
-   a macro and the \} terminating a conditional block, e.g.
-   .if n \{
-   .br \}
-   reported by ulrich spoerlein  Tue, 19 Oct 2010 20:39:50 +0200
  ************************************************************************
- * formatter bugs
- ************************************************************************
- ************************************************************************
  * missing features
  ************************************************************************
-Line 50
+Line 26
 Line 50
 Line 26
    .ad   -- re-enable adjustment without changing the mode
    Adjustment mode is ignored while in no-fill mode (.nf).
- - .it (line traps) occur in mysql(1), yasm_arch(7)
+ - .fc (field control)
-   generated by DocBook XSL Stylesheets v1.71.1 <http://docbook.sf.net/>
+   found by naddy@ in xloadimage(1)
-   reported by brad@  Sat, 15 Jan 2011 15:48:18 -0500
+ - .nr third argument (auto-increment step size, requires \n+)
+   found by bentley@ in sbcl(1)  Mon, 9 Dec 2013 18:36:57 -0700
  - .ns (no-space mode) occurs in xine-config(1)
    reported by brad@  Sat, 15 Jan 2011 15:45:23 -0500
- - xloadimage(1) wants .ti (temporary indent), rep by naddy@
  - .ta (tab settings) occurs in ircbug(1) and probably gnats(1)
    reported by brad@  Sat, 15 Jan 2011 15:50:51 -0500
+   also Tcl_NewStringObj(3) via wiz@  Wed, 5 Mar 2014 22:27:43 +0100
- - pod2man expects `tr' to be implemented for \*(-- to work
+ - .ti (temporary indent)
+   found by naddy@ in xloadimage(1)
+   found by bentley@ in nmh(1)  Mon, 23 Apr 2012 13:38:28 -0600
- - \c (interrupted text) occurs in chat(8)
+ - .while and .shift
+   found by jca@ in ratpoison(1)  Sun, 30 Jun 2013 12:01:09 +0200
+ - \c (interrupted text) should prevent the line break
+   even inside .Bd literal; that occurs in chat(8)
+   also found in cclive(1) - DocBook output
+ - \h horizontal move
+   found in cclive(1) DocBook output
+   Anthony J. Bentley on discuss@  Sat, 21 Sep 2013 22:29:34 -0600
+ - \n+ and \n- numerical register increment and decrement
+   found by bentley@ in sbcl(1)  Mon, 9 Dec 2013 18:36:57 -0700
+ - \w'' width measurements
+   would not be very useful without an expression parser, see below
+   needed for Tcl_NewStringObj(3) via wiz@  Wed, 5 Mar 2014 22:27:43 +0100
+ - using undefined strings or macros defines them to be empty
+   wl@  Mon, 14 Nov 2011 14:37:01 +0000
  --- missing mdoc features ----------------------------------------------
  - fix bad block nesting involving multiple identical explicit blocks
-Line 89
+Line 87
 Line 89
 Line 87
    because libmdoc does not yet use mandoc_getarg().
    Also check what happens in plain text, it must be identical to \e.
+ - .Bd -centered implies -filled, not -unfilled, which is not
+   easy to implement; it requires code similar to .ce, which
+   we don't have either.
+   Besides, groff has bug causing text right *before* .Bd -centered
+   to be centered as well.
  - .Bd -filled should not be the same as .Bd -ragged, but align both
    the left and right margin.  In groff, it is implemented in terms
    of .ad b, which we don't have either.  Found in cksum(1).
-Line 101
+Line 105
 Line 101
 Line 105
  - explicitly disallow nested `Bl -column', which would clobber internal
    flags defined for struct mdoc_macro
+ - In .Bl -column .It, the end of the line probably has to be regarded
+   as an implicit .Ta, if there could be one, see the following mildly
+   ugly code from login.conf(5):
+     .Bl -column minpasswordlen program xetcxmotd
+     .It path Ta path Ta value of Dv _PATH_DEFPATH
+     .br
+     Default search path.
+   reported by Michal Mazurek <akfaew at jasminek dot net>
+   via jmc@ Thu, 7 Apr 2011 16:00:53 +0059
  - inside `.Bl -column' phrases, punctuation is handled like normal
    text, e.g. `.Bl -column .It Fl x . Ta ...' should give "-x -."
-Line 112
+Line 126
 Line 112
 Line 126
  - have a blank `It' head for `Bl -tag' not puke
+ - check whether it is correct that `D1' uses INDENT+1;
+   does it need its own constant?
  - prohibit `Nm' from having non-text HEAD children
    (e.g., NetBSD mDNSShared/dns-sd.1)
    (mdoc_html.c and mdoc_term.c `Nm' handlers can be slightly simplified)
- --- missing man features -----------------------------------------------
+ - support translated section names
+   e.g. x11/scrotwm scrotwm_es.1:21:2: error: NAME section must be first
+   that one uses NOMBRE because it is spanish...
+   deraadt tends to think that section-dependent macro behaviour
+   is a bad idea in the first place, so this may be irrelevant
- - bashbug(1) complains "line scope broken" after
+ - When there is free text in the SYNOPSIS and that free text contains
-   .SM
+   the .Nm macro, groff somehow understands to treat the .Nm as an in-line
-   .B something
+   macro, while mandoc treats it as a block macro and breaks the line.
-   should either just work or be a warning
+   No idea how the logic for distinguishing in-line and block instances
-   reported by naddy@
+   should be, needs investigation.
+   uqs@  Thu, 2 Jun 2011 11:03:51 +0200
+   uqs@  Thu, 2 Jun 2011 11:33:35 +0200
- - groff an-ext.tmac macros (.UR, .UE) occur in xine(5)
+ --- missing man features -----------------------------------------------
-   reported by brad@  Sat, 15 Jan 2011 15:45:23 -0500
+ - -T[x]html doesn't stipulate non-collapsing spaces in literal mode
  --- missing tbl features -----------------------------------------------
- - implement basic non-parametric .de to support e.g. sox(1)
+ - look at the POSIX manuals in the books/man-pages-posix port,
-   reported by naddy@ Sat, 16 Oct 2010 23:51:57 +0200
+   they use some unsupported tbl(7) features.
-   *** sox(1) still doesn't work, tbl(1) errors need investigation
+ - investigate tbl(1) errors in sox(1)
+   see also naddy@  Sat, 16 Oct 2010 23:51:57 +0200
+ - allow standalone `.' to be interpreted as an end-of-layout
+   delimiter instead of being thrown away as a no-op roff line
+   reported by Yuri Pankov, Wed 18 May 2011 11:34:59 CEST
  --- missing misc features ----------------------------------------------
+ - italic correction (\/) in PostScript mode
+   Werner LEMBERG on groff at gnu dot org  Sun, 10 Nov 2013 12:47:46
+ - When makewhatis(8) encounters a FATAL parse error,
+   it silently treats the file as formatted, which makes no sense
+   at all for paths like man1/foo.1 - and which also contradicts
+   what the manual says at the end of the description.
+   The end result will be ENOENT for file names returned
+   by mansearch() in manpage.file.
+ - makewhatis(8) for preformatted pages:
+   parse the section number from the header line
+   and compare to the section number from the directory name
+ - Does makewhatis(8) detect missing NAME sections, missing names,
+   and missing descriptions in all the file formats?
  - clean up escape sequence handling, creating three classes:
    (1) fully implemented, or parsed and ignored without loss of content
    (2) unimplemented, potentially causing loss of content
-Line 142
+Line 189
 Line 142
 Line 189
        see textproc/mgdiff(1) for nice examples
    (3) undefined, just output the character -> perhaps WARNING
- - The \t escape sequence is the same as a literal tab, see for example
+ - kettenis wants base roff, ms, and me  Fri, 1 Jan 2010 22:13:15 +0100 (CET)
-   the ASCII table in hexdump(1) where
-     .Bl -column \&000_nu \&001_so \&002_st \&003_et \&004_eo
-     .It \&000\ nul\t001\ soh\t002\ stx\t003\ etx\t004\ eot\t005\ enq
-   produces
-nul  001 soh  002 stx  003 etx  004 eot  005 enq
-   and the example in oldrdist(1)
+ --- compatibility checks -----------------------------------------------
+ - is .Bk implemented correctly in modern groff?
+   sobrado@  Tue, 19 Apr 2011 22:12:55 +0200
+ - compare output to Heirloom roff, Solaris roff, and
+   http://repo.or.cz/w/neatroff.git  http://litcave.rudi.ir/
  - look at pages generated from reStructeredText, e.g. devel/mercurial hg(1)
    These are a weird mixture of man(7) and custom autogenerated low-level
    roff stuff.  Figure out to what extent we can cope.
-Line 157
+Line 206
 Line 157
 Line 206
    noted by stsp@  Sat, 24 Apr 2010 09:17:55 +0200
    reminded by nicm@  Mon, 3 May 2010 09:52:41 +0100
+ - look at pages generated from ronn(1) github.com/rtomayko/ronn
+   (based on markdown)
+ - look at pages generated from Texinfo source by yat2m, e.g. security/gnupg
+   First impression is not that bad.
+ - look at pages generated by pandoc; see
+   https://github.com/jgm/pandoc/blob/master/src/Text/Pandoc/Writers/Man.hs
+   porting planned by kili@  Thu, 19 Jun 2014 19:46:28 +0200
  - check compatibility with Plan9:
    http://swtch.com/usr/local/plan9/tmac/tmac.an
    http://swtch.com/plan9port/man/man7/man.html
    "Anthony J. Bentley" <anthonyjbentley@gmail.com> 28 Dec 2010 21:58:40 -0700
+ - check compatibility with the man(7) formatter
+   https://raw.githubusercontent.com/rofl0r/hardcore-utils/master/man.c
  ************************************************************************
  * formatting issues: ugly output
  ************************************************************************
-Line 169
+Line 231
 Line 169
 Line 231
  - a column list with blank `Ta' cells triggers a spurrious
    start-with-whitespace printing of a newline
- - double quotes inside double quotes are escaped by doubling them
-   implement this in mdoc(7), too
-   so far, we only have it in roff(7) and man(7)
-   reminded by millert@  Thu, 09 Dec 2010 17:29:52 -0500
- - perl(1) SYNOPSIS looks bad; reported by deraadt@
-) man(7) seems to need SYNOPSIS .Nm blocks, too
  - In .Bl -column,
    .It Em Authentication<tab>Key Length
    ought to render "Key Length" with emphasis, too,
    see OpenBSD iked.conf(5).
+   reported again Nicolas Joly via wiz@ Wed, 12 Oct 2011 00:20:00 +0200
  - empty phrases in .Bl column produce too few blanks
    try e.g. .Bl -column It Ta Ta
-Line 195
+Line 250
 Line 195
 Line 250
    of in_line() - put trailing punctuation out of scope.
    Found in mount_nfs(8) and exports(5), search for "Appendix".
+ - Trailing punctuation after .%T triggers EOS spacing, at least
+   outside .Rs (eek!).  Simply setting ARGSFL_DELIM for .%T is not
+   the right solution, it sends mandoc into an endless loop.
+   reported by Nicolas Joly  Sat, 17 Nov 2012 11:49:54 +0100
+ - global variables in the SYNOPSIS of section 3 pages
+   .Vt vs .Vt/.Va vs .Ft/.Va vs .Ft/.Fa ...
+   from kristaps@  Tue, 08 Jun 2010 11:13:32 +0200
  - in enclosures, mandoc sometimes fancies a bogus end of sentence
    reminded by jmc@  Thu, 23 Sep 2010 18:13:39 +0059
+ - formatting /usr/local/man/man1/latex2man.1 with groff and mandoc
+   reveals lots of bugs both in groff and mandoc...
+   reported by bentley@  Wed, 22 May 2013 23:49:30 -0600
+ --- PDF issues ---------------------------------------------------------
+ - PDF output doesn't use a monospaced font for .Bd -literal
+   Example: "mandoc -Tpdf afterboot.8 > output.pdf && pdfviewer output.pdf".
+   Search the text "Routing tables".
+   Also check what PostScript mode does when fixing this.
+   reported by juanfra@ Wed, 04 Jun 2014 21:44:58 +0200
+ --- HTML issues --------------------------------------------------------
+ - <dl><dt><dd> formatting is ugly
+   hints are easy to find on the web, e.g.
+   http://stackoverflow.com/questions/1713048/
+   see also matthew@  Fri, 18 Jul 2014 19:25:12 -0700
+ - check https://github.com/trentm/mdocml
  ************************************************************************
- * formatting issues: gratuitious differences
+ * formatting issues: gratuitous differences
  ************************************************************************
  - .Rv (and probably .Ex) print different text if an `Nm' has been named
-Line 210
+Line 295
 Line 210
 Line 295
    is just "o\bo".
    see for example OpenBSD ksh(1)
- - The characters "|" and "\*(Ba" should never be bold,
+ - In .Bl -enum -width 0n, groff continues one the same line after
-   not even in the middle of a word, e.g. ".Cm b\*(Bac" in
+   the number, mandoc breaks the line.
-   "mknod [-m mode] name b|c major minor"
+   mail to kristaps@  Mon, 20 Jul 2009 02:21:39 +0200
-   in OpenBSD ksh(1)
- - A bogus .Pp between two .It must not produce a double blank line,
+ - .Pp between two .It in .Bl -column should produce one,
-   see between -R and -r in OpenBSD rm(1), before "update" in mount(8),
+   not two blank lines, see e.g. login.conf(5).
-   or in DIAGNOSTICS in init(8), or before "is always true" in ksh(1).
+   reported by jmc@  Sun, 17 Apr 2011 14:04:58 +0059
-   The same happens with .Pp just before .El, see bgpd.conf(5).
+   reported again by sthen@  Wed, 18 Jan 2012 02:09:39 +0000 (UTC)
-   Also have `It' complain if `Pp' is invoked at certain times (not
-   -compact?).
  - If the *first* line after .It is .Pp, break the line right after
    the tag, do not pad with space characters before breaking.
-Line 230
+Line 312
 Line 230
 Line 312
    in between, see for example tmux(1).
    reported by nicm@  13 Jan 2011 00:18:57 +0000
+ - Trailing punctuation after .It should trigger EOS spacing.
+   reported by Nicolas Joly  Sat, 17 Nov 2012 11:49:54 +0100
+   Probably, this should be fixed somewhere in termp_it_pre(), not sure.
  - .Nx 1.0a
    should be "NetBSD 1.0A", not "NetBSD 1.0a",
    see OpenBSD ccdconfig(8).
-Line 243
+Line 329
 Line 243
 Line 329
      .Bl -tag -width ".Dv message"
    in magic(5), located in src/usr.bin/file, is the same
    as -width 7n, not -width 11n.
+   The same applies to .Bl -column column widths;
+   reported again by Nicolas Joly Thu, 1 Mar 2012 13:41:26 +0100 via wiz@ 5 Mar
+   reported again by Franco Fichtner Fri, 27 Sep 2013 21:02:28 +0200
+   An easy partial fix would be to just skip the first word if it starts
+   with a dot, including any following white space, when measuring.
  - The \& zero-width character counts as output.
    That is, when it is alone on a line between two .Pp,
    we want three blank lines, not two as in mandoc.
- - When .Fn arguments exceed one output line, all but the first
-   should be indented, see e.g. rpc(3);
-   reported by jmc@ on discuss@  Fri, 29 Oct 2010 13:48:33 +0100
- - .Ns should work when called at the end of an input line, see
-   the following code in vi(1):
-     .It Xo
-     .Op Ar line
-     .Cm a Ns Op Cm ppend Ns
-     .Op Cm !\&
-     .Xc
-     The input text is appended after the specified line.
  - Header lines of excessive length:
    Port OpenBSD man_term.c rev. 1.25 to mdoc_term.c
    and document it in mdoc(7) and man(7) COMPATIBILITY
    found while talking to Chris Bennett
- - In man(7), the sequence
-     .HP
-     one line of regular text
-     .SH
-   should not produce two blank lines before the .SH,
-   see for example named-checkconf(8).
- - In man(7), the sequence
-     regular text
-     .RS
-     .IP
-   should have a blank line between the text and the beginning of the
-   indented paragraph, see for example sudo(1).
- - At least sometimes, .sp is ignored right after .SH; see named.conf(5).
  - trailing whitespace must be ignored even when followed by a font escape,
    see for example
      makes
-Line 290
+Line 352
 Line 290
 Line 352
    in dig(1).
  ************************************************************************
- * error reporting issues
+ * warning issues
  ************************************************************************
+ - check that MANDOCERR_BADTAB is thrown in the right cases,
+   i.e. when finding a literal tab character in fill mode,
+   and possibly change the wording of the warning message
+   to refer to fill mode, not literal mode
+   See the mail from Werner LEMBERG on the groff list,
+   Fri, 14 Feb 2014 18:54:42 +0100 (CET)
+ - warn about "new sentence, new line"
+ - mandoc_special does not really check the escape sequence,
+   but just the overall format
+ - integrate mdoclint into mandoc ("end-of-line whitespace" thread)
+   from jmc@  Mon, 13 Jul 2009 17:12:09 +0100
+   from kristaps@  Mon, 13 Jul 2009 18:34:53 +0200
+   from jmc@  Mon, 13 Jul 2009 17:45:37 +0059
+   from kristaps@  Mon, 13 Jul 2009 19:02:03 +0200
+ - -Tlint parser errors and warnings to stdout
+   to tech@mdocml, naddy@  Wed, 28 Sep 2011 11:21:46 +0200
+   wait!  kristaps@  Sun, 02 Oct 2011 17:12:52 +0200
+ - for system errors, use errno/strerror/warn/err
  ************************************************************************
+ * documentation issues
+ ************************************************************************
+ - mention hyphenation rules:
+   breaking at letter-letter in text mode (not macro args)
+   proper hyphenation is unimplemented
+ - talk about spacing around delimiters
+   to jmc@, kristaps@  Sat, 23 Apr 2011 17:41:27 +0200
+ - mark macros as: page structure domain, manual domain, general text domain
+   is this useful?
+ - mention /usr/share/misc/mdoc.template in mdoc(7)?
+ ************************************************************************
  * performance issues
  ************************************************************************
+ - Why are we using MAP_SHARED, not MAP_PRIVATE for mmap(2)?
+   How does SQLITE_CONFIG_PAGECACHE actually work?  Document it!
+   from kristaps@  Sat, 09 Aug 2014 13:51:36 +0200
  Several areas can be cleaned up to make mandoc even faster.  These are
  - improve hashing mechanism for macros (quite important: performance)
-Line 305  Several areas can be cleaned up to make mandoc even fa
+Line 411  Several areas can be cleaned up to make mandoc even fa
 Line 305  Several areas can be cleaned up to make mandoc even fa
 Line 411  Several areas can be cleaned up to make mandoc even fa
  - the PDF file is HUGE: this can be reduced by using relative offsets
+ - instead of re-initialising the roff predefined-strings set before each
+   parse, create a read-only version the first time and copy it
  ************************************************************************
  * structural issues
  ************************************************************************
+ - We use the input line number at several places to distinguish
+   same-line from different-line input.  That plainly doesn't work
+   with user-defined macros, leading to random breakage.
  - Find better ways to prevent endless loops
    in roff(7) macro and string expansion.
+ - Finish cleanup of date handling.
+   Decide which formats should be recognized where.
+   Update both mdoc(7) and man(7) documentation.
+   Triggered by  Tim van der Molen  Tue, 22 Feb 2011 20:30:45 +0100
+ - Consider creating some views that will make the database more
+   readable from the sqlite3 shell.  Consider using them to
+   abstract from the database structure, too.
+   suggested by espie@  Sat, 19 Apr 2014 14:52:57 +0200

CVSweb