Page MenuHomeFreeBSD
Differential D53224

quot: Make nusers unsigned
ClosedPublic
Actions

Authored by des on Mon, Oct 20, 5:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 23, 7:17 AM
Unknown Object (File)
Thu, Oct 23, 7:17 AM
Unknown Object (File)
Thu, Oct 23, 7:17 AM
Unknown Object (File)
Thu, Oct 23, 7:15 AM
Unknown Object (File)
Thu, Oct 23, 5:27 AM
Unknown Object (File)
Wed, Oct 22, 11:02 PM
Unknown Object (File)
Tue, Oct 21, 5:40 AM
Unknown Object (File)
Tue, Oct 21, 12:43 AM
View All 12 Files
Subscribers
imp

Details

Reviewers
rlibby
Commits
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.Mon, Oct 20, 5:55 PM
Herald added a subscriber: imp. · View Herald TranscriptMon, Oct 20, 5:55 PM
des requested review of this revision.Mon, Oct 20, 5:55 PM
Harbormaster completed remote builds in B67945: Diff 164641.Mon, Oct 20, 5:55 PM
rlibby accepted this revision.Mon, Oct 20, 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.Mon, Oct 20, 7:21 PM
des added inline comments.Mon, Oct 20, 9:41 PM
usr.sbin/quot/quot.c
194

See D53130

Closed by commit rGfa347930f89d: quot: Make nusers unsigned (authored by des). · Explain WhyMon, Oct 20, 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.

Revision Contents
Changeset List

PathSize
usr.sbin/
quot/
27 lines

Diff 164659

View Options

usr.sbin/quot/quot.c

Loading...