Page MenuHomeFreeBSD
Differential D37247

ping: Remove a vestigial notdef
ClosedPublic
Actions

Authored by jlduran_gmail.com on Nov 3 2022, 9:52 AM.
Tags
None
Referenced Files
F97644596: D37247.diff
Mon, Sep 30, 1:39 PM
Unknown Object (File)
Tue, Sep 17, 6:03 AM
Unknown Object (File)
Mon, Sep 16, 11:06 PM
Unknown Object (File)
Mon, Sep 16, 10:20 PM
Unknown Object (File)
Sun, Sep 15, 10:01 PM
Unknown Object (File)
Mon, Sep 9, 4:11 AM
Unknown Object (File)
Thu, Sep 5, 2:06 PM
Unknown Object (File)
Jul 31 2024, 3:17 AM
View All 30 Files
Subscribers
imp
Contributor Reviews (src)

Details

Reviewers
markj
Group Reviewers
Contributor Reviews (src)
Commits
rG294cb8bb3acc: ping: Remove a vestigial notdef
rG909e2e1b6c34: ping: Remove a vestigial notdef
Summary

It was once a function on 4.3BSD, pr_type() [1], used to convert an ICMP "type" field to a printable string. In 4.4BSD it was superseded by pr_icmph() [2].

NetBSD [3] and OpenBSD [4] have already removed it.

[1]: https://minnie.tuhs.org/cgi-bin/utree.pl?file=4.3BSD/usr/src/etc/ping.c
[2]: https://minnie.tuhs.org/cgi-bin/utree.pl?file=4.4BSD/usr/src/sbin/ping/ping.c
[3]: https://github.com/NetBSD/src/commit/203dfd34867991fd002f747d74a96f26ae80d41c
[4]: https://github.com/openbsd/src/commit/9bbbbbb75d24e3d166ddd0cb6cb4417b78561309

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jlduran_gmail.com created this revision.Nov 3 2022, 9:52 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 3 2022, 9:52 AM
jlduran_gmail.com requested review of this revision.Nov 3 2022, 9:52 AM
jlduran_gmail.com edited the summary of this revision. (Show Details)Nov 3 2022, 10:14 AM
markj accepted this revision as: markj.Nov 3 2022, 1:06 PM
This revision is now accepted and ready to land.Nov 3 2022, 1:06 PM
markj added a comment.Nov 3 2022, 1:39 PM

Could I ask you to make your git commits available somehow, either by email or in a pull request?

jlduran_gmail.com added a comment.Nov 3 2022, 1:51 PM
In D37247#845961, @markj wrote:

Could I ask you to make your git commits available somehow, either by email or in a pull request?

Absolutely!
https://github.com/freebsd/freebsd-src/pull/624

Closed by commit rG909e2e1b6c34: ping: Remove a vestigial notdef (authored by jlduran_gmail.com, committed by markj). · Explain WhyNov 3 2022, 2:41 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG909e2e1b6c34: ping: Remove a vestigial notdef.
markj added a commit: rG294cb8bb3acc: ping: Remove a vestigial notdef.Nov 10 2022, 5:04 PM

Revision Contents
Changeset List

PathSize
sbin/
ping/
16 lines

Diff 112542

View Options

sbin/ping/ping.c

Loading...