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)
	Thu, Mar 19, 3:31 AM

	Unknown Object (File)
	Wed, Mar 18, 3:46 AM

	Unknown Object (File)
	Mon, Mar 16, 6:59 PM

	Unknown Object (File)
	Sun, Mar 15, 4:25 PM

	Unknown Object (File)
	Sun, Mar 15, 9:34 AM

	Unknown Object (File)
	Sat, Mar 14, 7:34 PM

	Unknown Object (File)
	Sat, Mar 14, 2:15 AM

	Unknown Object (File)
	Fri, Mar 13, 11:33 PM

View All 55 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...