Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace incorrect text with ix macro #7

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

N0NB
Copy link

@N0NB N0NB commented Aug 22, 2023

Extra text was found in the output that matched other uses of the 'ix' macro. Replace '129' with '.ix' to fix.

The 'mmt' command is now present in the index.

N0NB added 4 commits August 20, 2023 12:09
Extra text was found in the output that matched other uses of the 'ix'
macro.  Replace '129' with '.ix' to fix.

The 'mmt' command is now present in the index.
The value of register Hb controls the heading levels that will have a
line break after the heading.  The addition of " lines" implies that
the value controls the number of blank lines which is not the case per
original AT&T MM documentation:

https://github.com/n-t-roff/DWB3.3/blob/562bb8615116f46077ca03e8561323cc7a89673e/doc/mm/mm.04#L257

Unfortunately, this erroneous text also appears in the original UTP and
confuses the intended use of the register.
The sentence preceeding the description of legal values correctly notes
'1' as the default.  Showing '0' and '1' as 'default' is confusing.  The
available AT&T MM manual confirms the default is '1':

https://github.com/n-t-roff/DWB3.3/blob/562bb8615116f46077ca03e8561323cc7a89673e/doc/mm/mm.04#L282

This extreneous string appears in the original UTP.
The HF example only sets bold for the first four headings while the
explanatory text below states that the first five headings will be in
bold type.  Correct the example to match the text.

This inconsistency is present in the original UTP.
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