Skip to content

docs: fix invalid \fi font escape in man page (should be \fI)#903

Open
greymoth-jp wants to merge 1 commit into
sharkdp:masterfrom
greymoth-jp:greymoth-jp/docs/hyperfine-manpage-italic-escape
Open

docs: fix invalid \fi font escape in man page (should be \fI)#903
greymoth-jp wants to merge 1 commit into
sharkdp:masterfrom
greymoth-jp:greymoth-jp/docs/hyperfine-manpage-italic-escape

Conversation

@greymoth-jp

Copy link
Copy Markdown

What

Two typos in doc/hyperfine.1:

  1. \fi\fI (lines 312, 314)\fi is not a valid nroff/troff font escape. Italic is \fI (uppercase I). Every other italic span in the file already uses \fI (\fINUM\fP, \fIFILE\fP, \fIUNIT\fP, …). The two \fiNAME\fP occurrences in the --command-name option description would render as literal text instead of an italic word.

  2. --time-unit--time\-unit in SYNOPSIS (line 38) — minor consistency fix. Every other hyphenated long option in the SYNOPSIS block escapes its internal hyphens (\-\-min\-runs, \-\-max\-runs, \-\-parameter\-scan, etc.). \-\-time-unit had an unescaped second hyphen.

Diff

-.RB [ \-\-time-unit
+.RB [ \-\-time\-unit

-\fB\-n\fR, \fB\-\-command\-name\fR \fiNAME\fP
+\fB\-n\fR, \fB\-\-command\-name\fR \fINAME\fP

-Give a meaningful \fiNAME\fP to a command.
+Give a meaningful \fINAME\fP to a command.

No functional or behavioural change — documentation only.

Two occurrences of \`\fi\` (lines 312/314, --command-name option) are not
valid nroff/troff escape sequences; italic is \`\fI\` (uppercase I). All
other italic spans in the file correctly use \`\fI\`. Fix both.

Also correct \`\-\-time-unit\` in the SYNOPSIS (line 38) to
\`\-\-time\-unit\` for consistency with every other hyphenated long option
in that block (e.g. \`\-\-min\-runs\`, \`\-\-max\-runs\`).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant