Page MenuHomeFreeBSD

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

Authored by melifaro on May 20 2023, 11:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 24, 12:35 AM
Unknown Object (File)
Apr 14 2025, 8:48 AM
Unknown Object (File)
Apr 14 2025, 4:51 AM
Unknown Object (File)
Apr 14 2025, 1:24 AM
Unknown Object (File)
Apr 14 2025, 1:22 AM
Unknown Object (File)
Apr 8 2025, 11:16 AM
Unknown Object (File)
Apr 8 2025, 8:02 AM
Unknown Object (File)
Mar 23 2025, 9:18 PM
Subscribers

Diff Detail

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

Event Timeline

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 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.