Page MenuHomeFreeBSD
Differential D47132

tests: Fix style of netpfil/common/divapp.c
ClosedPublic
Actions

Authored by igoro on Oct 15 2024, 5:24 PM.
Tags
None
Referenced Files
F152209673: D47132.id.diff
Mon, Apr 13, 10:46 AM
Unknown Object (File)
Tue, Apr 7, 8:10 AM
Unknown Object (File)
Mon, Mar 23, 6:14 AM
Unknown Object (File)
Mon, Mar 16, 6:21 PM
Unknown Object (File)
Mar 5 2026, 8:28 PM
Unknown Object (File)
Feb 20 2026, 4:59 AM
Unknown Object (File)
Feb 17 2026, 12:34 PM
Unknown Object (File)
Feb 7 2026, 7:09 PM
View All 88 Files
Subscribers
glebius
imp
melifaro
vegeta_tuxpowered.net

Details

Reviewers
kp
markj
Commits
rGb3e7d4b67ce6: tests: Fix style of netpfil/common/divapp.c
Summary

No functional change intended.

Diff Detail

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

Event Timeline

igoro created this revision.Oct 15 2024, 5:24 PM
Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro, imp. · View Herald TranscriptOct 15 2024, 5:24 PM
igoro requested review of this revision.Oct 15 2024, 5:24 PM
Harbormaster completed remote builds in B60015: Diff 144934.Oct 15 2024, 5:24 PM
markj added inline comments.Oct 15 2024, 8:27 PM
tests/sys/netpfil/common/divapp.c
104

If you use err() instead of errx(), information from errno will be appended automatically.

igoro updated this revision to Diff 144984.Oct 16 2024, 10:27 AM

Use the standard way for error reporting

Harbormaster completed remote builds in B60033: Diff 144984.Oct 16 2024, 10:27 AM
igoro marked an inline comment as done.Oct 16 2024, 10:28 AM
igoro added inline comments.
tests/sys/netpfil/common/divapp.c
104

Nice, thanks.

markj accepted this revision.Oct 16 2024, 1:02 PM

Approved.

This revision is now accepted and ready to land.Oct 16 2024, 1:02 PM
Closed by commit rGb3e7d4b67ce6: tests: Fix style of netpfil/common/divapp.c (authored by igoro). · Explain WhyOct 16 2024, 4:26 PM
This revision was automatically updated to reflect the committed changes.
igoro marked an inline comment as done.
igoro added a commit: rGb3e7d4b67ce6: tests: Fix style of netpfil/common/divapp.c.

Revision Contents
Changeset List

PathSize
tests/
sys/
netpfil/
common/
13 lines

Diff 145000

View Options

tests/sys/netpfil/common/divapp.c

Loading...