Page MenuHomeFreeBSD
Differential D40180

ifconfig: simplify ifconfig() by factoring out ifa add/del logic
ClosedPublic
Actions

Authored by melifaro on May 20 2023, 11:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 19, 2:47 PM
Unknown Object (File)
Sat, Aug 15, 11:25 AM
Unknown Object (File)
Mon, Aug 10, 9:52 AM
Unknown Object (File)
Sun, Aug 9, 11:32 PM
Unknown Object (File)
Sun, Aug 9, 5:18 PM
Unknown Object (File)
Jul 24 2026, 9:23 PM
Unknown Object (File)
Jul 21 2026, 6:21 AM
Unknown Object (File)
Jul 18 2026, 3:25 PM
View All Files
Subscribers
imp

Details

Reviewers
kp
Group Reviewers
network
Commits
rG63ad5ce7335b: ifconfig: simplify ifconfig() by factoring out ifa add/del logic

Diff Detail

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

Event Timeline

melifaro created this revision.May 20 2023, 11:23 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 20 2023, 11:23 AM
melifaro requested review of this revision.May 20 2023, 11:23 AM
Harbormaster completed remote builds in B51576: Diff 122160.May 20 2023, 11:23 AM
melifaro added a parent revision: D40179: ifconfig: split argument parsing.May 20 2023, 11:24 AM
melifaro edited the summary of this revision. (Show Details)
melifaro added reviewers: network, kp.
melifaro added a child revision: D40182: ifconfig: switch IPv4/IPv6 address manipulations to Netlink.May 20 2023, 11:54 AM
kp accepted this revision as: kp.May 20 2023, 3:39 PM
kp added inline comments.
sbin/ifconfig/ifconfig.c
977

Not blocking here, but we really need to reduce the number of global variables in ifconfig.

983

I think style(9) still requires that declaration to be at the top of a block.

This revision is now accepted and ready to land.May 20 2023, 3:39 PM
melifaro marked an inline comment as done.May 21 2023, 9:42 AM
melifaro added inline comments.
sbin/ifconfig/ifconfig.c
977

I agree - that's why I started their merge under args. This diff has a specific scope of simplifying ifconfig() function and it doesn't change anything w.r.t the global variable handling.

Closed by commit rG63ad5ce7335b: ifconfig: simplify ifconfig() by factoring out ifa add/del logic (authored by melifaro). · Explain WhyMay 21 2023, 9:43 AM
This revision was automatically updated to reflect the committed changes.
melifaro added a commit: rG63ad5ce7335b: ifconfig: simplify ifconfig() by factoring out ifa add/del logic.

Revision Contents
Changeset List

PathSize
sbin/
ifconfig/
83 lines

Diff 122181

View Options

sbin/ifconfig/ifconfig.c

Loading...