Page MenuHomeFreeBSD

bpf(4): Update the manual page to reflect reality

Authored by gbe on Aug 7 2020, 3:32 PM.
Referenced Files
Unknown Object (File)
Wed, Jan 18, 9:54 PM
Unknown Object (File)
Sun, Jan 8, 11:02 PM
Unknown Object (File)
Sun, Jan 8, 12:35 PM
Unknown Object (File)
Wed, Jan 4, 5:06 PM
Unknown Object (File)
Dec 15 2022, 12:20 AM
Unknown Object (File)
Dec 13 2022, 12:51 AM



bpf(4): Update the manual page to reflect reality

PR 131918

Test Plan

mandoc output review and 'mandoc -Tlint' checks

Diff Detail

rS FreeBSD src repository - subversion
Lint Not Applicable
Tests Not Applicable

Event Timeline

gbe requested review of this revision.Aug 7 2020, 3:32 PM
gbe created this revision.

Some minor style comments...

687 ↗(On Diff #75567)

Is this early line break necessary here? I think you call pull the next line up to this one.

700 ↗(On Diff #75567)

Another early line break...

The summary of this review is lacking detail

101 ↗(On Diff #75567)

Where did this information go? Should the list of of protocols be extended rather than deleted? Is data not still in network byte order and one should be using byteorder to extract values?

700 ↗(On Diff #75567)

Actually a wrong line break, the line break should be at the comma (,) after affect.

  • Fix two early line breaks.
gbe marked 3 inline comments as done.Aug 24 2020, 8:06 AM
101 ↗(On Diff #75567)

The author of the original patch has deleted the information. I currently have not checked the code what protocols are supported. Since the original PR is from 2009, I invited network to have look.

I can of bring the paragraph back and extent it if necessary.

Could someone review the differential?

This revision is now accepted and ready to land.Oct 9 2020, 2:51 PM
This revision was automatically updated to reflect the committed changes.