Page MenuHomeFreeBSD

nfsstat: Complete libxo transition.
ClosedPublic

Authored by yanhaowang on Aug 11 2023, 1:45 PM.
Tags
None
Referenced Files
F107579648: D41428.diff
Thu, Jan 16, 4:37 AM
Unknown Object (File)
Sun, Jan 12, 1:57 AM
Unknown Object (File)
Nov 24 2024, 1:41 AM
Unknown Object (File)
Nov 7 2024, 1:49 PM
Unknown Object (File)
Oct 29 2024, 11:37 AM
Unknown Object (File)
Oct 22 2024, 6:57 PM
Unknown Object (File)
Oct 22 2024, 6:57 PM
Unknown Object (File)
Oct 20 2024, 1:59 PM

Diff Detail

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

Event Timeline

usr.bin/nfsstat/nfsstat.c
86

Please avoid contiguous blank lines.

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

This comment was removed by yanhaowang.
yanhaowang added inline comments.
usr.bin/nfsstat/nfsstat.c
507–508

Ignore this, xo_err can't replace fprintf.

  • Use xo_error instead of fprintf.
  • Decide using 1 and 0 instead of EXIT_FAILURE and EXIT_SUCESS.
This revision is now accepted and ready to land.Aug 19 2023, 1:34 PM
This revision was automatically updated to reflect the committed changes.