kqueue.2: Fix reference to unexisting kevent1()
ClosedPublic
Actions

Authored by tembun_bk.ru on Sat, Jan 24, 10:45 AM.

Details

Reviewers

kib
jhb
ziaee
dab

Group Reviewers

docs
manpages
Contributor Reviews (src)

Commits

rG4efe75af0869: kqueue.2: Fix reference to unexisting kevent1()

Summary

kqueue1() was meant.

While here, make the wording in the AUTHORS section more clear.

Fixes: 9b1585384d53 ("kqueue.2: Editorial pass")
PR: 291908

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tembun_bk.ru created this revision.Sat, Jan 24, 10:45 AM

Herald added a subscriber: imp. · View Herald TranscriptSat, Jan 24, 10:45 AM

tembun_bk.ru requested review of this revision.Sat, Jan 24, 10:45 AM

markj added a subscriber: markj.Sat, Jan 24, 2:11 PM

markj added inline comments.

lib/libsys/kqueue.2
979–980	I suspect this is deliberate. Maybe "kqueue subsystem" would be more clear.

tembun_bk.ru added inline comments.Sat, Jan 24, 2:15 PM

lib/libsys/kqueue.2
979–980	I also thought about it. I agree, 'subsystem' will be more clear and unambiguous.

Make the wording in the AUTHORS section more clear.

tembun_bk.ru marked an inline comment as done.Sat, Jan 24, 2:25 PM

kib added inline comments.Sat, Jan 24, 4:21 PM

lib/libsys/kqueue.2
978–979	It is strange to use function name markup for the subsystem name then. Perhaps use an emphasis, or remove the markup at all.

Use .Em instead of .Fn for the subsystem name.

tembun_bk.ru marked an inline comment as done.Sat, Jan 24, 6:50 PM

kib accepted this revision.Sat, Jan 24, 7:34 PM

This revision is now accepted and ready to land.Sat, Jan 24, 7:34 PM

dab accepted this revision.Sun, Jan 25, 10:40 PM

Closed by commit rG4efe75af0869: kqueue.2: Fix reference to unexisting kevent1() (authored by tembun_bk.ru, committed by markj). · Explain WhySun, Jan 25, 10:42 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG4efe75af0869: kqueue.2: Fix reference to unexisting kevent1().

Revision Contents
Changeset List

Path

Size

lib/

libsys/

kqueue.2

8 lines

Diff 170420

View Options

kqueue.2: Fix reference to unexisting kevent1()ClosedPublicActions