Page MenuHomeFreeBSD
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)
Fri, May 22, 8:56 PM
Unknown Object (File)
Sun, May 17, 7:00 AM
Unknown Object (File)
Fri, May 15, 2:23 PM
Unknown Object (File)
Fri, May 15, 11:18 AM
Unknown Object (File)
Fri, May 15, 7:20 AM
Unknown Object (File)
Thu, May 14, 11:30 PM
Unknown Object (File)
Wed, May 13, 6:22 PM
Unknown Object (File)
Wed, May 13, 5:58 PM
View All 88 Files
Subscribers
imp

Details

Reviewers
rlibby
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
rlibby accepted this revision.Oct 20 2025, 7:21 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

PathSize
usr.sbin/
quot/
27 lines

Diff 164659

View Options

usr.sbin/quot/quot.c

Loading...