Update to reflect pipe() calling pipe2().
ClosedPublic
Actions

Authored by brooks on Jun 23 2016, 10:18 PM.

Details

Reviewers

jhb
wblock

Group Reviewers

manpages

Commits

rS303104: Update to reflect the fact that pipe() is a wrapper around the pipe2()

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

brooks updated this revision to Diff 17837.Jun 23 2016, 10:18 PM

brooks retitled this revision from to Update to reflect pipe() calling pipe2()..

brooks updated this object.

brooks edited the test plan for this revision. (Show Details)

brooks added a reviewer: jhb.

Herald added a reviewer: manpages. · View Herald TranscriptJun 23 2016, 10:18 PM

Herald added a subscriber: imp. · View Herald Transcript

Thanks.

We should document this for open() as well then.

This revision is now accepted and ready to land.Jun 23 2016, 10:52 PM

wblock added a subscriber: wblock.Jun 24 2016, 7:33 PM

wblock added inline comments.

lib/libc/sys/pipe.2
121 ↗	(On Diff #17837)	Should dtrace(1) and ktrace(1) be added to the SEE ALSO section? I don't know if we have a hard rule about that.

brooks added inline comments.Jun 24 2016, 9:13 PM

lib/libc/sys/pipe.2
121 ↗	(On Diff #17837)	I waffled on that myself, but went with not adding them because they are effectively unrelated to pipe().

wblock accepted this revision.Jun 24 2016, 10:21 PM

wblock added a reviewer: wblock.

wblock added inline comments.

lib/libc/sys/pipe.2
121 ↗	(On Diff #17837)	Works for me. Thanks!

Closed by commit rS303104: Update to reflect the fact that pipe() is a wrapper around the pipe2() (authored by brooks). · Explain WhyJul 20 2016, 6:02 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

lib/

libc/

sys/

pipe.2

21 lines

Diff 18605

View Options

Update to reflect pipe() calling pipe2().ClosedPublicActions