Page MenuHomeFreeBSD
Differential D48594

ctld: Use bool in uclparse.c
ClosedPublic
Actions

Authored by jhb on Jan 22 2025, 7:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 23, 4:07 AM
Unknown Object (File)
Tue, Feb 18, 9:57 AM
Unknown Object (File)
Sat, Feb 15, 12:23 AM
Unknown Object (File)
Fri, Feb 14, 10:57 PM
Unknown Object (File)
Thu, Feb 13, 11:02 PM
Unknown Object (File)
Thu, Feb 13, 6:59 PM
Unknown Object (File)
Mon, Feb 10, 11:04 AM
Unknown Object (File)
Mon, Feb 10, 10:12 AM
View All 16 Files
Subscribers
imp

Details

Reviewers
trasz
asomers
mav
Commits
rG78cd9f0c0941: ctld: Use bool in uclparse.c
Summary

Previously these functions all returned either 0 or 1, but 0 was
success and 1 was failure. Use a bool where false indicates failure
instead which is more readable.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.Jan 22 2025, 7:52 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 22 2025, 7:52 PM
jhb requested review of this revision.Jan 22 2025, 7:52 PM
jhb added a parent revision: D48593: iscsi: Move valid_iscsi_name to libiscsiutil.
jhb added a child revision: D48595: ctld: Use nvlist instead of home-rolled name-value lists.
Harbormaster completed remote builds in B61887: Diff 149718.Jan 22 2025, 7:53 PM
asomers accepted this revision.Jan 22 2025, 8:04 PM
This revision is now accepted and ready to land.Jan 22 2025, 8:04 PM
mav accepted this revision.Jan 23 2025, 4:26 PM

I guess instead of 1 could be expected some error codes, that just never happened.

jhb updated this revision to Diff 149825.Jan 23 2025, 7:14 PM

Two more that I missed

This revision now requires review to proceed.Jan 23 2025, 7:14 PM
Harbormaster completed remote builds in B61945: Diff 149825.Jan 23 2025, 7:14 PM
asomers accepted this revision.Jan 23 2025, 7:15 PM
This revision is now accepted and ready to land.Jan 23 2025, 7:15 PM
Closed by commit rG78cd9f0c0941: ctld: Use bool in uclparse.c (authored by jhb). · Explain WhyJan 24 2025, 2:55 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG78cd9f0c0941: ctld: Use bool in uclparse.c.

Revision Contents
Changeset List

PathSize
usr.sbin/
ctld/
256 lines

Diff 149883

View Options

usr.sbin/ctld/uclparse.c

Loading...