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
F82866483: D40180.id122181.diff
Fri, May 3, 8:54 AM
F82866474: D40180.id.diff
Fri, May 3, 8:54 AM
F82861085: D40180.id122160.diff
Fri, May 3, 7:35 AM
F82854769: D40180.diff
Fri, May 3, 5:55 AM
Unknown Object (File)
Sat, Apr 27, 3:39 PM
Unknown Object (File)
Sat, Apr 27, 6:19 AM
Unknown Object (File)
Tue, Apr 23, 5:37 PM
Unknown Object (File)
Mar 21 2024, 9:19 PM
View All 35 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 Passed
Unit
No Test Coverage
Build Status
Buildable 51576
Build 48467: arc lint + arc unit

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 122160

View Options

sbin/ifconfig/ifconfig.c

Loading...