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
Unknown Object (File)
Sat, Oct 25, 3:33 AM
Unknown Object (File)
Wed, Oct 22, 11:11 PM
Unknown Object (File)
Wed, Oct 22, 11:29 AM
Unknown Object (File)
Sat, Oct 18, 11:23 AM
Unknown Object (File)
Tue, Oct 14, 10:51 PM
Unknown Object (File)
Tue, Oct 14, 10:51 PM
Unknown Object (File)
Tue, Oct 14, 10:51 PM
Unknown Object (File)
Tue, Oct 14, 10:51 PM
View All 61 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...