Paths

Table of Contentst

Differential D53224

quot: Make nusers unsigned
ClosedPublic
Actions

Authored by des on Oct 20 2025, 5:55 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 19, 10:14 AM

	Unknown Object (File)
	Sat, Apr 18, 6:09 AM

	Unknown Object (File)
	Sat, Apr 18, 5:13 AM

	Unknown Object (File)
	Sat, Apr 18, 4:38 AM

	Unknown Object (File)
	Sat, Apr 18, 4:37 AM

	Unknown Object (File)
	Thu, Apr 9, 3:52 AM

	Unknown Object (File)
	Tue, Apr 7, 8:07 AM

	Unknown Object (File)
	Sun, Apr 5, 1:11 AM

View All 64 Files

Subscribers

Details

Reviewers

Commits

rG201038a01ba8: quot: Make nusers unsigned
rG2427ae41b7cf: quot: Make nusers unsigned
rGfa347930f89d: quot: Make nusers unsigned

Summary

Fixes: 195b00ec45e5 ("quot: Clean up")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Oct 20 2025, 5:55 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 20 2025, 5:55 PM

des requested review of this revision.Oct 20 2025, 5:55 PM

Harbormaster completed remote builds in B67945: Diff 164641.Oct 20 2025, 5:55 PM

Thanks for the follow up.

usr.sbin/quot/quot.c
194	This is good but consider if you prefer a more conventional `for (usr = users, i = 0; i < nusers; usr++, i++)` or similar here. Similar question for other loop counters.
212	I see that there's more of a reason to count backward here.
374	Did you want to give `n` and this line the same treatment?

This revision is now accepted and ready to land.Oct 20 2025, 7:21 PM

des added inline comments.Oct 20 2025, 9:41 PM

usr.sbin/quot/quot.c
194	See D53130

Closed by commit rGfa347930f89d: quot: Make nusers unsigned (authored by des). · Explain WhyOct 20 2025, 9:42 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGfa347930f89d: quot: Make nusers unsigned.

des added a commit: rG2427ae41b7cf: quot: Make nusers unsigned.

cperciva added a commit: rG201038a01ba8: quot: Make nusers unsigned.Oct 30 2025, 11:53 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

quot/

27 lines

Diff 164659

usr.sbin/quot/quot.c

Loading...