Page MenuHomeFreeBSD
Differential D41428

nfsstat: Complete libxo transition.
ClosedPublic
Actions

Authored by yanhaowang on Aug 11 2023, 1:45 PM.
Tags
None
Referenced Files
F134014424: D41428.id126224.diff
Thu, Oct 30, 1:05 AM
F134006912: D41428.id125879.diff
Wed, Oct 29, 11:45 PM
F134005948: D41428.id126224.diff
Wed, Oct 29, 11:34 PM
F134004426: D41428.id126225.diff
Wed, Oct 29, 11:16 PM
Unknown Object (File)
Thu, Oct 23, 2:52 AM
Unknown Object (File)
Wed, Oct 22, 1:08 PM
Unknown Object (File)
Sat, Oct 18, 11:28 PM
Unknown Object (File)
Tue, Oct 14, 5:06 AM
View All 100 Files
Subscribers
guest-patmaddox
imp

Details

Reviewers
lwhsu
des
Commits
rGb698093da430: nfsstat(1): Complete libxo transition

Diff Detail

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

Event Timeline

yanhaowang created this revision.Aug 11 2023, 1:45 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 11 2023, 1:45 PM
yanhaowang requested review of this revision.Aug 11 2023, 1:45 PM
Harbormaster completed remote builds in B53050: Diff 125879.Aug 11 2023, 1:45 PM
yanhaowang added a reviewer: des.Aug 11 2023, 1:46 PM
lwhsu added inline comments.Aug 11 2023, 7:44 PM
usr.bin/nfsstat/nfsstat.c
86

Please avoid contiguous blank lines.

yanhaowang updated this revision to Diff 125895.Aug 12 2023, 1:29 AM

Remove unnecessary space.

Harbormaster completed remote builds in B53058: Diff 125895.Aug 12 2023, 1:29 AM
yanhaowang marked an inline comment as done.Aug 12 2023, 1:30 AM
des requested changes to this revision.Aug 17 2023, 4:02 PM
des added inline comments.
usr.bin/nfsstat/nfsstat.c
507–508

This should be xo_error().

This revision now requires changes to proceed.Aug 17 2023, 4:02 PM
yanhaowang added inline comments.Aug 19 2023, 8:14 AM
usr.bin/nfsstat/nfsstat.c
507–508

Maybe use xo_err(1, "usage: nfsstat [-cdemszW] [-w wait]\n"); to replace the fprintf and exit function?

yanhaowang updated this revision to Diff 126224.Aug 19 2023, 8:36 AM
This comment was removed by yanhaowang.
Harbormaster completed remote builds in B53172: Diff 126224.Aug 19 2023, 8:36 AM
yanhaowang marked an inline comment as done.Aug 19 2023, 8:42 AM
yanhaowang added inline comments.
usr.bin/nfsstat/nfsstat.c
507–508

Ignore this, xo_err can't replace fprintf.

yanhaowang updated this revision to Diff 126225.EditedAug 19 2023, 8:46 AM
  • Use xo_error instead of fprintf.
  • Decide using 1 and 0 instead of EXIT_FAILURE and EXIT_SUCESS.
Harbormaster completed remote builds in B53173: Diff 126225.Aug 19 2023, 8:46 AM
des accepted this revision.Aug 19 2023, 1:34 PM
This revision is now accepted and ready to land.Aug 19 2023, 1:34 PM
guest-patmaddox added a subscriber: guest-patmaddox.Oct 19 2023, 9:46 PM
Closed by commit rGb698093da430: nfsstat(1): Complete libxo transition (authored by yanhaowang, committed by lwhsu). · Explain WhyOct 12 2024, 6:30 PM
This revision was automatically updated to reflect the committed changes.
lwhsu added a commit: rGb698093da430: nfsstat(1): Complete libxo transition.

Revision Contents
Changeset List

PathSize
usr.bin/
nfsstat/
13 lines

Diff 144739

View Options

usr.bin/nfsstat/nfsstat.c

Loading...