Page MenuHomeFreeBSD
Differential D31090

math(3): function args must not use .Ar, use .Fa instead
ClosedPublic
Actions

Authored by gbe on Jul 7 2021, 11:53 AM.
Tags
None
Referenced Files
F149510636: D31090.diff
Tue, Mar 24, 10:56 PM
F149406097: D31090.id91935.diff
Tue, Mar 24, 6:22 AM
F149406088: D31090.id91935.diff
Tue, Mar 24, 6:22 AM
Unknown Object (File)
Mon, Mar 23, 9:28 PM
Unknown Object (File)
Mon, Mar 23, 4:19 AM
Unknown Object (File)
Sun, Mar 22, 8:37 AM
Unknown Object (File)
Mon, Mar 16, 5:13 AM
Unknown Object (File)
Sun, Mar 15, 10:53 PM
View All Files
Subscribers
imp

Details

Reviewers
imp
Group Reviewers
manpages
Commits
rG9428c3c912db: math(3): Use the .Fa macro for function arguments
rGc5cbef2f85e6: math(3): Use the .Fa macro for function arguments
Summary
  • function arguments must not use the .Ar macro, so use the .Fa macro instead
Test Plan

mandoc output review and 'mandoc -Tlint' checks

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

gbe created this revision.Jul 7 2021, 11:53 AM
Herald added a subscriber: imp. · View Herald TranscriptJul 7 2021, 11:53 AM
gbe requested review of this revision.Jul 7 2021, 11:53 AM
Harbormaster completed remote builds in B40366: Diff 91935.Jul 7 2021, 11:53 AM
imp accepted this revision.Jul 7 2021, 6:59 PM

Change looks good. The commit message would maybe be better to say that .Ar is used for command line arguments and .Fa is used for function arguments...

This revision is now accepted and ready to land.Jul 7 2021, 6:59 PM
Closed by commit rGc5cbef2f85e6: math(3): Use the .Fa macro for function arguments (authored by gbe). · Explain WhyJul 9 2021, 5:56 PM
This revision was automatically updated to reflect the committed changes.
gbe added a commit: rGc5cbef2f85e6: math(3): Use the .Fa macro for function arguments.
gbe added a commit: rG9428c3c912db: math(3): Use the .Fa macro for function arguments.Jul 12 2021, 4:08 AM

Revision Contents
Changeset List

PathSize
lib/
msun/
man/
2 lines
4 lines
2 lines
4 lines
16 lines
6 lines
2 lines
2 lines
8 lines
6 lines
4 lines

Diff 92022

View Options

lib/msun/man/acos.3

Loading...
View Options

lib/msun/man/acosh.3

Loading...
View Options

lib/msun/man/asin.3

Loading...
View Options

lib/msun/man/asinh.3

Loading...
View Options

lib/msun/man/atan2.3

Loading...
View Options

lib/msun/man/atanh.3

Loading...
View Options

lib/msun/man/ccos.3

Loading...
View Options

lib/msun/man/ccosh.3

Loading...
View Options

lib/msun/man/exp.3

Loading...
View Options

lib/msun/man/sincos.3

Loading...
View Options

lib/msun/man/sqrt.3

Loading...