Page MenuHomeFreeBSD
Differential D54457

df: Cleanup and refactor
Needs ReviewPublic
Actions

Authored by jlduran on Jan 2 2026, 7:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 22 2026, 10:51 PM
Unknown Object (File)
Jan 20 2026, 9:05 AM
Unknown Object (File)
Jan 19 2026, 9:10 PM
Unknown Object (File)
Jan 17 2026, 7:38 PM
Unknown Object (File)
Jan 16 2026, 7:11 PM
Unknown Object (File)
Jan 14 2026, 11:26 AM
Unknown Object (File)
Jan 11 2026, 9:46 PM
Unknown Object (File)
Jan 9 2026, 8:25 PM
View All 14 Files
Subscribers
imp

Details

Reviewers
des
Summary

Improve code style and type safety:

  • Change boolean flags from int to bool type for clarity
  • Given sys/param.h is already included, use the MAX() macro it provides
  • Update variable types from int to size_t
  • Adjust string comparison logic for better readability
  • Various style(9) fixes

No functional changes intended.

Suggested by: des

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jlduran created this revision.Jan 2 2026, 7:56 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 2 2026, 7:56 PM
jlduran requested review of this revision.Jan 2 2026, 7:56 PM
Harbormaster completed remote builds in B69573: Diff 168896.Jan 2 2026, 7:56 PM
jlduran added a child revision: D54264: df: ceil() the percentage value.Jan 2 2026, 8:00 PM
jlduran added a parent revision: D54016: df: Import tests from NetBSD.
jlduran added a reviewer: des.Jan 2 2026, 8:05 PM
jlduran mentioned this in D54265: df: Fix the header when using the -P option.Jan 2 2026, 8:17 PM
jlduran updated this revision to Diff 168931.Jan 3 2026, 2:11 AM
  • Remove stray reference to availheader
des added inline comments.Sat, Feb 28, 10:27 AM
bin/df/df.c
60–66
285
305
466–467
483–484
588–589
603–608
614–617

otherwise it returns 0 for 0

jlduran updated this revision to Diff 172973.Sun, Mar 1, 6:18 PM
jlduran marked 8 inline comments as done.
jlduran edited the summary of this revision. (Show Details)

Address suggestions

des added inline comments.Sun, Mar 1, 7:23 PM
bin/df/df.c
38–41
86–87

these should all be bool, you'll have to change makevfslist() and checkvfsname() to suit.

163

I'm not sure lflag is actually needed, we can just check whether vfslist_l is null like we do for vfslist_t.

295–296

this lets you drop the cast in the malloc() call below

308
309–310

this should be fatal, imo

369

should mntsize be size_t?

371–372
493–496
614–617

actually I was mistaken, I missed that the if above used <= instead of <... either revert this or change the if (and update the comment).

jlduran updated this revision to Diff 172986.Sun, Mar 1, 9:37 PM
jlduran marked 9 inline comments as done.
jlduran edited the summary of this revision. (Show Details)

Address suggestions.

bin/df/df.c
309–310

Will be addressed in a separate revision, as it is a functional change.

jlduran added a child revision: D55610: df: Make malloc() failure fatal.Sun, Mar 1, 9:54 PM
jlduran marked an inline comment as done.
jlduran added inline comments.
bin/df/df.c
309–310

Addressed in https://reviews.freebsd.org/D55610

Revision Contents
Changeset List

PathSize
bin/
df/
54 lines

Diff 168931

View Options

bin/df/df.c

Loading...