Paths

Table of Contentst

Differential D45441

Summary: Migrate from printb to print_bits for locally defined bit fields.
ClosedPublic
Actions

Authored by gordon on Jun 1 2024, 3:57 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Feb 10, 12:28 PM

	Unknown Object (File)
	Sat, Feb 8, 7:59 PM

	Unknown Object (File)
	Sat, Feb 8, 4:03 AM

	Unknown Object (File)
	Thu, Feb 6, 11:13 PM

	Unknown Object (File)
	Mon, Feb 3, 11:54 PM

	Unknown Object (File)
	Dec 4 2024, 5:30 AM

	Unknown Object (File)
	Nov 27 2024, 5:50 PM

	Unknown Object (File)
	Nov 27 2024, 5:49 PM

View All 57 Files

Subscribers

Details

Reviewers

emaste
jhb

Commits

rGa68e4f7a0652: Migrate from printb to print_bits for locally defined bit fields.
rGc3e9423743d9: Move print_bits to ifconfig.c and make available to other src files.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

gordon created this revision.Jun 1 2024, 3:57 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 1 2024, 3:58 PM

gordon requested review of this revision.Jun 1 2024, 3:58 PM

Harbormaster completed remote builds in B57991: Diff 139306.Jun 1 2024, 3:59 PM

emaste added inline comments.Jun 1 2024, 4:36 PM

sbin/ifconfig/af_nd6.c
56 ↗	(On Diff #139306)
66–67 ↗	(On Diff #139306)	you could move the `#ifdef` to just make these two conditional now?
181 ↗	(On Diff #139306)	If refactoring I'd drop this pseduo-bit-15 hack here and just add a `if (isdefif) printf("...")` assuming `isdefif` is in fact boolean and there isn't some magic here

gordon marked 2 inline comments as done.Jun 3 2024, 3:02 PM

gordon added inline comments.

sbin/ifconfig/af_nd6.c
181 ↗	(On Diff #139306)	I don't think I'm going to try to change the logic here. At least not with this change.

Address comments from emaste.

Harbormaster completed remote builds in B58015: Diff 139353.Jun 3 2024, 3:05 PM

emaste accepted this revision.Jun 3 2024, 3:16 PM

This revision is now accepted and ready to land.Jun 3 2024, 3:16 PM

Move print_bits to ifconfig.c and make available to other src files.
Migrate from printb to print_bits for locally defined bit fields.

This revision now requires review to proceed.Jun 3 2024, 3:18 PM

Harbormaster completed remote builds in B58016: Diff 139354.Jun 3 2024, 3:18 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 3 2024, 3:52 PM

Closed by commit rGc3e9423743d9: Move print_bits to ifconfig.c and make available to other src files. (authored by gordon). · Explain Why

This revision was automatically updated to reflect the committed changes.

gordon added a commit: rGc3e9423743d9: Move print_bits to ifconfig.c and make available to other src files..

gordon added a commit: rGa68e4f7a0652: Migrate from printb to print_bits for locally defined bit fields..

Revision Contents
Changeset List

Path

Size

sbin/

ifconfig/

2 lines

23 lines

ifconfig_netlink.c

23 lines

Diff 139356

sbin/ifconfig/ifconfig.h

Loading...

sbin/ifconfig/ifconfig.c

Loading...

sbin/ifconfig/ifconfig_netlink.c

Loading...