Annotation of mandoc/TODO, Revision 1.183
1.1 kristaps 1: ************************************************************************
1.37 kristaps 2: * Official mandoc TODO.
1.183 ! schwarze 3: * $Id: TODO,v 1.182 2014/10/10 10:49:38 schwarze Exp $
1.27 kristaps 4: ************************************************************************
1.145 schwarze 5:
6: ************************************************************************
7: * crashes
8: ************************************************************************
9:
1.169 schwarze 10: - The abort() in bufcat(), html.c, can be triggered via buffmt_includes()
11: by running -Thtml -Oincludes on a file containing a long .In argument.
12: Fixing this will probably require reworking the whole bufcat() concept.
1.101 schwarze 13:
1.61 schwarze 14: ************************************************************************
1.1 kristaps 15: * missing features
16: ************************************************************************
1.70 kristaps 17:
1.79 schwarze 18: --- missing roff features ----------------------------------------------
19:
1.82 schwarze 20: - .ad (adjust margins)
21: .ad l -- adjust left margin only (flush left)
22: .ad r -- adjust right margin only (flush right)
23: .ad c -- center text on line
24: .ad b -- adjust both margins (alias: .ad n)
25: .na -- temporarily disable adjustment without changing the mode
26: .ad -- re-enable adjustment without changing the mode
27: Adjustment mode is ignored while in no-fill mode (.nf).
1.150 schwarze 28:
29: - .fc (field control)
30: found by naddy@ in xloadimage(1)
31:
1.160 schwarze 32: - .nr third argument (auto-increment step size, requires \n+)
33: found by bentley@ in sbcl(1) Mon, 9 Dec 2013 18:36:57 -0700
34:
1.80 schwarze 35: - .ns (no-space mode) occurs in xine-config(1)
36: reported by brad@ Sat, 15 Jan 2011 15:45:23 -0500
1.61 schwarze 37:
1.81 schwarze 38: - .ta (tab settings) occurs in ircbug(1) and probably gnats(1)
39: reported by brad@ Sat, 15 Jan 2011 15:50:51 -0500
1.167 schwarze 40: also Tcl_NewStringObj(3) via wiz@ Wed, 5 Mar 2014 22:27:43 +0100
1.150 schwarze 41:
42: - .ti (temporary indent)
43: found by naddy@ in xloadimage(1)
44: found by bentley@ in nmh(1) Mon, 23 Apr 2012 13:38:28 -0600
1.154 schwarze 45:
46: - .while and .shift
47: found by jca@ in ratpoison(1) Sun, 30 Jun 2013 12:01:09 +0200
1.79 schwarze 48:
1.132 schwarze 49: - \c (interrupted text) should prevent the line break
50: even inside .Bd literal; that occurs in chat(8)
1.155 schwarze 51: also found in cclive(1) - DocBook output
52:
53: - \h horizontal move
54: found in cclive(1) DocBook output
55: Anthony J. Bentley on discuss@ Sat, 21 Sep 2013 22:29:34 -0600
1.125 schwarze 56:
1.160 schwarze 57: - \n+ and \n- numerical register increment and decrement
58: found by bentley@ in sbcl(1) Mon, 9 Dec 2013 18:36:57 -0700
59:
1.167 schwarze 60: - \w'' width measurements
61: would not be very useful without an expression parser, see below
62: needed for Tcl_NewStringObj(3) via wiz@ Wed, 5 Mar 2014 22:27:43 +0100
63:
1.125 schwarze 64: - using undefined strings or macros defines them to be empty
65: wl@ Mon, 14 Nov 2011 14:37:01 +0000
1.167 schwarze 66:
1.79 schwarze 67: --- missing mdoc features ----------------------------------------------
68:
1.18 schwarze 69: - fix bad block nesting involving multiple identical explicit blocks
70: see the OpenBSD mdoc_macro.c 1.47 commit message
71:
1.1 kristaps 72: - .Bl -column .Xo support is missing
73: ultimate goal:
74: restore .Xr and .Dv to
75: lib/libc/compat-43/sigvec.3
76: lib/libc/gen/signal.3
77: lib/libc/sys/sigaction.2
78:
1.28 schwarze 79: - edge case: decide how to deal with blk_full bad nesting, e.g.
80: .Sh .Nm .Bk .Nm .Ek .Sh found by jmc@ in ssh-keygen(1)
81: from jmc@ Wed, 14 Jul 2010 18:10:32 +0100
82:
1.74 schwarze 83: - \\ is now implemented correctly
84: * when defining strings and macros using .ds and .de
85: * when parsing roff(7) and man(7) macro arguments
86: It does not yet work in mdoc(7) macro arguments
87: because libmdoc does not yet use mandoc_getarg().
88: Also check what happens in plain text, it must be identical to \e.
1.82 schwarze 89:
1.174 schwarze 90: - .Bd -centered implies -filled, not -unfilled, which is not
91: easy to implement; it requires code similar to .ce, which
92: we don't have either.
93: Besides, groff has bug causing text right *before* .Bd -centered
94: to be centered as well.
95:
1.82 schwarze 96: - .Bd -filled should not be the same as .Bd -ragged, but align both
97: the left and right margin. In groff, it is implemented in terms
98: of .ad b, which we don't have either. Found in cksum(1).
1.22 schwarze 99:
1.10 kristaps 100: - implement blank `Bl -column', such as
101: .Bl -column
102: .It foo Ta bar
103: .El
1.11 kristaps 104:
105: - explicitly disallow nested `Bl -column', which would clobber internal
106: flags defined for struct mdoc_macro
1.102 schwarze 107:
108: - In .Bl -column .It, the end of the line probably has to be regarded
109: as an implicit .Ta, if there could be one, see the following mildly
110: ugly code from login.conf(5):
111: .Bl -column minpasswordlen program xetcxmotd
112: .It path Ta path Ta value of Dv _PATH_DEFPATH
113: .br
114: Default search path.
115: reported by Michal Mazurek <akfaew at jasminek dot net>
116: via jmc@ Thu, 7 Apr 2011 16:00:53 +0059
1.42 schwarze 117:
118: - inside `.Bl -column' phrases, punctuation is handled like normal
119: text, e.g. `.Bl -column .It Fl x . Ta ...' should give "-x -."
120:
121: - inside `.Bl -column' phrases, TERMP_IGNDELIM handling by `Pf'
122: is not safe, e.g. `.Bl -column .It Pf a b .' gives "ab."
123: but should give "ab ."
1.12 kristaps 124:
125: - set a meaningful default if no `Bl' list type is assigned
1.13 kristaps 126:
127: - have a blank `It' head for `Bl -tag' not puke
1.20 kristaps 128:
1.174 schwarze 129: - check whether it is correct that `D1' uses INDENT+1;
130: does it need its own constant?
131:
1.20 kristaps 132: - prohibit `Nm' from having non-text HEAD children
133: (e.g., NetBSD mDNSShared/dns-sd.1)
134: (mdoc_html.c and mdoc_term.c `Nm' handlers can be slightly simplified)
1.109 schwarze 135:
1.174 schwarze 136: - support translated section names
137: e.g. x11/scrotwm scrotwm_es.1:21:2: error: NAME section must be first
138: that one uses NOMBRE because it is spanish...
139: deraadt tends to think that section-dependent macro behaviour
140: is a bad idea in the first place, so this may be irrelevant
141:
1.109 schwarze 142: - When there is free text in the SYNOPSIS and that free text contains
143: the .Nm macro, groff somehow understands to treat the .Nm as an in-line
144: macro, while mandoc treats it as a block macro and breaks the line.
145: No idea how the logic for distinguishing in-line and block instances
146: should be, needs investigation.
147: uqs@ Thu, 2 Jun 2011 11:03:51 +0200
148: uqs@ Thu, 2 Jun 2011 11:33:35 +0200
1.57 kristaps 149:
1.79 schwarze 150: --- missing man features -----------------------------------------------
151:
1.119 kristaps 152: - -T[x]html doesn't stipulate non-collapsing spaces in literal mode
1.80 schwarze 153:
1.79 schwarze 154: --- missing tbl features -----------------------------------------------
155:
1.171 schwarze 156: - look at the POSIX manuals in the books/man-pages-posix port,
157: they use some unsupported tbl(7) features.
158:
1.183 ! schwarze 159: - use Unicode U+2500 to U+256C for table borders
! 160: in tbl(7) -Tutf-8 output
! 161: suggested by bentley@ Tue, 14 Oct 2014 04:10:55 -0600
1.105 kristaps 162:
163: - allow standalone `.' to be interpreted as an end-of-layout
164: delimiter instead of being thrown away as a no-op roff line
165: reported by Yuri Pankov, Wed 18 May 2011 11:34:59 CEST
1.79 schwarze 166:
1.182 schwarze 167: --- missing eqn features -----------------------------------------------
168:
169: - set, delim, fonts
170:
171: - The "size" keyword is parsed, but ignored by the formatter.
172:
173: - The spacing characters `~', `^', and tab are currently ignored,
174: see User's Guide (Second Edition) page 2 section 4.
175:
176: - Mark and lineup are parsed and ignored,
177: see User's Guide (Second Edition) page 5 section 15.
178:
1.79 schwarze 179: --- missing misc features ----------------------------------------------
1.159 schwarze 180:
181: - italic correction (\/) in PostScript mode
182: Werner LEMBERG on groff at gnu dot org Sun, 10 Nov 2013 12:47:46
1.153 schwarze 183:
1.173 schwarze 184: - When makewhatis(8) encounters a FATAL parse error,
185: it silently treats the file as formatted, which makes no sense
186: at all for paths like man1/foo.1 - and which also contradicts
187: what the manual says at the end of the description.
188: The end result will be ENOENT for file names returned
189: by mansearch() in manpage.file.
190:
1.171 schwarze 191: - makewhatis(8) for preformatted pages:
192: parse the section number from the header line
193: and compare to the section number from the directory name
194:
195: - Does makewhatis(8) detect missing NAME sections, missing names,
196: and missing descriptions in all the file formats?
197:
1.79 schwarze 198: - clean up escape sequence handling, creating three classes:
199: (1) fully implemented, or parsed and ignored without loss of content
200: (2) unimplemented, potentially causing loss of content
201: or serious mangling of formatting (e.g. \n) -> ERROR
202: see textproc/mgdiff(1) for nice examples
203: (3) undefined, just output the character -> perhaps WARNING
1.83 schwarze 204:
1.174 schwarze 205: - kettenis wants base roff, ms, and me Fri, 1 Jan 2010 22:13:15 +0100 (CET)
206:
207: --- compatibility checks -----------------------------------------------
208:
209: - is .Bk implemented correctly in modern groff?
210: sobrado@ Tue, 19 Apr 2011 22:12:55 +0200
211:
1.175 schwarze 212: - compare output to Heirloom roff, Solaris roff, and
213: http://repo.or.cz/w/neatroff.git http://litcave.rudi.ir/
1.178 schwarze 214:
215: - look at AT&T DWB http://www2.research.att.com/sw/download
216: Carsten Kunze <carsten dot kunze at arcor dot de> has patches
217: Mon, 4 Aug 2014 17:01:28 +0200
1.174 schwarze 218:
1.79 schwarze 219: - look at pages generated from reStructeredText, e.g. devel/mercurial hg(1)
220: These are a weird mixture of man(7) and custom autogenerated low-level
221: roff stuff. Figure out to what extent we can cope.
1.80 schwarze 222: For details, see http://docutils.sourceforge.net/rst.html
1.79 schwarze 223: noted by stsp@ Sat, 24 Apr 2010 09:17:55 +0200
224: reminded by nicm@ Mon, 3 May 2010 09:52:41 +0100
1.160 schwarze 225:
1.174 schwarze 226: - look at pages generated from ronn(1) github.com/rtomayko/ronn
227: (based on markdown)
228:
1.160 schwarze 229: - look at pages generated from Texinfo source by yat2m, e.g. security/gnupg
230: First impression is not that bad.
1.172 schwarze 231:
232: - look at pages generated by pandoc; see
233: https://github.com/jgm/pandoc/blob/master/src/Text/Pandoc/Writers/Man.hs
234: porting planned by kili@ Thu, 19 Jun 2014 19:46:28 +0200
1.71 schwarze 235:
236: - check compatibility with Plan9:
237: http://swtch.com/usr/local/plan9/tmac/tmac.an
238: http://swtch.com/plan9port/man/man7/man.html
239: "Anthony J. Bentley" <anthonyjbentley@gmail.com> 28 Dec 2010 21:58:40 -0700
1.63 schwarze 240:
1.174 schwarze 241: - check compatibility with the man(7) formatter
242: https://raw.githubusercontent.com/rofl0r/hardcore-utils/master/man.c
243:
1.1 kristaps 244: ************************************************************************
245: * formatting issues: ugly output
246: ************************************************************************
1.87 kristaps 247:
248: - a column list with blank `Ta' cells triggers a spurrious
249: start-with-whitespace printing of a newline
1.1 kristaps 250:
1.39 schwarze 251: - In .Bl -column,
252: .It Em Authentication<tab>Key Length
253: ought to render "Key Length" with emphasis, too,
254: see OpenBSD iked.conf(5).
1.123 schwarze 255: reported again Nicolas Joly via wiz@ Wed, 12 Oct 2011 00:20:00 +0200
1.39 schwarze 256:
1.1 kristaps 257: - empty phrases in .Bl column produce too few blanks
258: try e.g. .Bl -column It Ta Ta
259: reported by millert Fri, 02 Apr 2010 16:13:46 -0400
1.48 schwarze 260:
1.83 schwarze 261: - .%T can have trailing punctuation. Currently, it puts the trailing
262: punctuation into a trailing MDOC_TEXT element inside its own scope.
263: That element should rather be outside its scope, such that the
264: punctuation does not get underlines. This is not trivial to
265: implement because .%T then needs some features of in_line_eoln() -
266: slurp all arguments into one single text element - and one feature
267: of in_line() - put trailing punctuation out of scope.
268: Found in mount_nfs(8) and exports(5), search for "Appendix".
269:
1.147 schwarze 270: - Trailing punctuation after .%T triggers EOS spacing, at least
271: outside .Rs (eek!). Simply setting ARGSFL_DELIM for .%T is not
272: the right solution, it sends mandoc into an endless loop.
273: reported by Nicolas Joly Sat, 17 Nov 2012 11:49:54 +0100
274:
1.174 schwarze 275: - global variables in the SYNOPSIS of section 3 pages
276: .Vt vs .Vt/.Va vs .Ft/.Va vs .Ft/.Fa ...
277: from kristaps@ Tue, 08 Jun 2010 11:13:32 +0200
278:
1.48 schwarze 279: - in enclosures, mandoc sometimes fancies a bogus end of sentence
280: reminded by jmc@ Thu, 23 Sep 2010 18:13:39 +0059
1.149 schwarze 281:
282: - formatting /usr/local/man/man1/latex2man.1 with groff and mandoc
283: reveals lots of bugs both in groff and mandoc...
284: reported by bentley@ Wed, 22 May 2013 23:49:30 -0600
1.171 schwarze 285:
286: --- PDF issues ---------------------------------------------------------
287:
288: - PDF output doesn't use a monospaced font for .Bd -literal
289: Example: "mandoc -Tpdf afterboot.8 > output.pdf && pdfviewer output.pdf".
290: Search the text "Routing tables".
291: Also check what PostScript mode does when fixing this.
292: reported by juanfra@ Wed, 04 Jun 2014 21:44:58 +0200
1.173 schwarze 293:
294: --- HTML issues --------------------------------------------------------
295:
1.174 schwarze 296: - <dl><dt><dd> formatting is ugly
297: hints are easy to find on the web, e.g.
298: http://stackoverflow.com/questions/1713048/
299: see also matthew@ Fri, 18 Jul 2014 19:25:12 -0700
1.180 schwarze 300:
301: - The tables used to render the three-part page headers actually force
302: the width of the <body> to the max-width given for <html>.
303: Not yet sure how to fix that...
304: Observed by an Anonymous Coward on undeadly.org:
305: http://undeadly.org/cgi?action=article&sid=20140925064244&pid=1
1.177 schwarze 306:
307: - consider whether <var> can be used for Ar Dv Er Ev Fa Va.
308: from bentley@ Wed, 13 Aug 2014 09:17:55 -0600
1.174 schwarze 309:
1.173 schwarze 310: - check https://github.com/trentm/mdocml
1.35 kristaps 311:
1.182 schwarze 312: --- eqn issues ---------------------------------------------------------
313:
314: - If .EQ follows preceding text, a space should be output between the
315: text and the equation.
316:
1.1 kristaps 317: ************************************************************************
1.104 kristaps 318: * formatting issues: gratuitous differences
1.1 kristaps 319: ************************************************************************
1.65 kristaps 320:
321: - .Rv (and probably .Ex) print different text if an `Nm' has been named
322: or not (run a manual without `Nm blah' to see this). I'm not sure
323: that this exists in the wild, but it's still an error.
1.38 schwarze 324:
325: - In .Bl -bullet, the groff bullet is "+\b+\bo\bo", the mandoc bullet
326: is just "o\bo".
327: see for example OpenBSD ksh(1)
1.103 schwarze 328:
1.174 schwarze 329: - In .Bl -enum -width 0n, groff continues one the same line after
330: the number, mandoc breaks the line.
331: mail to kristaps@ Mon, 20 Jul 2009 02:21:39 +0200
332:
1.103 schwarze 333: - .Pp between two .It in .Bl -column should produce one,
334: not two blank lines, see e.g. login.conf(5).
335: reported by jmc@ Sun, 17 Apr 2011 14:04:58 +0059
1.129 schwarze 336: reported again by sthen@ Wed, 18 Jan 2012 02:09:39 +0000 (UTC)
1.39 schwarze 337:
1.83 schwarze 338: - If the *first* line after .It is .Pp, break the line right after
339: the tag, do not pad with space characters before breaking.
340: See the description of the a, c, and i commands in sed(1).
341:
342: - If the first line after .It is .D1, do not assert a blank line
343: in between, see for example tmux(1).
344: reported by nicm@ 13 Jan 2011 00:18:57 +0000
1.147 schwarze 345:
346: - Trailing punctuation after .It should trigger EOS spacing.
347: reported by Nicolas Joly Sat, 17 Nov 2012 11:49:54 +0100
348: Probably, this should be fixed somewhere in termp_it_pre(), not sure.
1.83 schwarze 349:
1.39 schwarze 350: - .Nx 1.0a
351: should be "NetBSD 1.0A", not "NetBSD 1.0a",
352: see OpenBSD ccdconfig(8).
1.83 schwarze 353:
1.39 schwarze 354: - In .Bl -tag, if a tag exceeds the right margin and must be continued
355: on the next line, it must be indented by -width, not width+1;
356: see "rule block|pass" in OpenBSD ifconfig(8).
1.56 schwarze 357:
1.83 schwarze 358: - When the -width string contains macros, the macros must be rendered
359: before measuring the width, for example
360: .Bl -tag -width ".Dv message"
361: in magic(5), located in src/usr.bin/file, is the same
362: as -width 7n, not -width 11n.
1.129 schwarze 363: The same applies to .Bl -column column widths;
364: reported again by Nicolas Joly Thu, 1 Mar 2012 13:41:26 +0100 via wiz@ 5 Mar
1.157 schwarze 365: reported again by Franco Fichtner Fri, 27 Sep 2013 21:02:28 +0200
366: An easy partial fix would be to just skip the first word if it starts
367: with a dot, including any following white space, when measuring.
1.83 schwarze 368:
1.56 schwarze 369: - The \& zero-width character counts as output.
370: That is, when it is alone on a line between two .Pp,
371: we want three blank lines, not two as in mandoc.
1.83 schwarze 372:
1.64 schwarze 373: - Header lines of excessive length:
374: Port OpenBSD man_term.c rev. 1.25 to mdoc_term.c
375: and document it in mdoc(7) and man(7) COMPATIBILITY
376: found while talking to Chris Bennett
1.83 schwarze 377:
378: - trailing whitespace must be ignored even when followed by a font escape,
379: see for example
380: makes
381: \fBdig \fR
382: operate in batch mode
383: in dig(1).
1.166 schwarze 384:
385: ************************************************************************
386: * warning issues
387: ************************************************************************
388:
389: - check that MANDOCERR_BADTAB is thrown in the right cases,
390: i.e. when finding a literal tab character in fill mode,
391: and possibly change the wording of the warning message
392: to refer to fill mode, not literal mode
393: See the mail from Werner LEMBERG on the groff list,
394: Fri, 14 Feb 2014 18:54:42 +0100 (CET)
1.174 schwarze 395:
396: - warn about "new sentence, new line"
397:
398: - mandoc_special does not really check the escape sequence,
399: but just the overall format
400:
401: - integrate mdoclint into mandoc ("end-of-line whitespace" thread)
402: from jmc@ Mon, 13 Jul 2009 17:12:09 +0100
403: from kristaps@ Mon, 13 Jul 2009 18:34:53 +0200
404: from jmc@ Mon, 13 Jul 2009 17:45:37 +0059
405: from kristaps@ Mon, 13 Jul 2009 19:02:03 +0200
406:
407: - -Tlint parser errors and warnings to stdout
408: to tech@mdocml, naddy@ Wed, 28 Sep 2011 11:21:46 +0200
409: wait! kristaps@ Sun, 02 Oct 2011 17:12:52 +0200
410:
411: - for system errors, use errno/strerror/warn/err
412:
413: ************************************************************************
414: * documentation issues
415: ************************************************************************
416:
417: - mention hyphenation rules:
418: breaking at letter-letter in text mode (not macro args)
419: proper hyphenation is unimplemented
420:
421: - talk about spacing around delimiters
422: to jmc@, kristaps@ Sat, 23 Apr 2011 17:41:27 +0200
423:
424: - mark macros as: page structure domain, manual domain, general text domain
425: is this useful?
426:
427: - mention /usr/share/misc/mdoc.template in mdoc(7)?
1.1 kristaps 428:
1.9 kristaps 429: ************************************************************************
430: * performance issues
431: ************************************************************************
1.176 schwarze 432:
433: - Why are we using MAP_SHARED, not MAP_PRIVATE for mmap(2)?
434: How does SQLITE_CONFIG_PAGECACHE actually work? Document it!
435: from kristaps@ Sat, 09 Aug 2014 13:51:36 +0200
1.9 kristaps 436:
437: Several areas can be cleaned up to make mandoc even faster. These are
438:
439: - improve hashing mechanism for macros (quite important: performance)
440:
441: - improve hashing mechanism for characters (not as important)
1.23 kristaps 442:
1.37 kristaps 443: - the PDF file is HUGE: this can be reduced by using relative offsets
1.115 kristaps 444:
445: - instead of re-initialising the roff predefined-strings set before each
446: parse, create a read-only version the first time and copy it
1.37 kristaps 447:
1.23 kristaps 448: ************************************************************************
449: * structural issues
450: ************************************************************************
1.122 schwarze 451:
452: - We use the input line number at several places to distinguish
453: same-line from different-line input. That plainly doesn't work
454: with user-defined macros, leading to random breakage.
1.67 schwarze 455:
456: - Find better ways to prevent endless loops
457: in roff(7) macro and string expansion.
1.91 schwarze 458:
1.96 schwarze 459: - Finish cleanup of date handling.
1.91 schwarze 460: Decide which formats should be recognized where.
461: Update both mdoc(7) and man(7) documentation.
462: Triggered by Tim van der Molen Tue, 22 Feb 2011 20:30:45 +0100
1.170 schwarze 463:
464: - Consider creating some views that will make the database more
465: readable from the sqlite3 shell. Consider using them to
466: abstract from the database structure, too.
467: suggested by espie@ Sat, 19 Apr 2014 14:52:57 +0200
468:
1.179 kristaps 469: ************************************************************************
470: * CGI issues
471: ************************************************************************
472:
473: - Enable HTTP compression by detecting gzip encoding and filtering
474: output through libz.
475: - Sandbox (see OpenSSH).
476: - Enable caching support via HTTP 304 and If-Modified-Since.
477: - Allow for cgi.h to be overridden by CGI environment variables.
478: Otherwise, binary distributions will inherit the compile-time
479: behaviour, which is not optimal.
480: - Have Mac OSX systems automatically disable -static compilation of the
481: CGI: -static isn't supported.
1.181 kristaps 482:
CVSweb