Paths

Table of Contentst

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

Use bool instead of int
style(9) fixes
Given sys/param.h is already included, use the MAX() macro it provides

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

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

Revision Contents
Changeset List

Path

Size

bin/

df/

df.c

54 lines

Diff 168931

View Options

df: Cleanup and refactorNeeds ReviewPublicActions