ctld: Use bool in uclparse.c
ClosedPublic
Actions

Authored by jhb on Jan 22 2025, 7:52 PM.

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

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

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

Path

Size

usr.sbin/

ctld/

uclparse.c

256 lines

Diff 149883

View Options

ctld: Use bool in uclparse.cClosedPublicActions