Page MenuHomeFreeBSD

efivar: use bool for booleans
ClosedPublic

Authored by imp on Apr 7 2021, 5:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 15, 8:44 PM
Unknown Object (File)
Fri, Mar 15, 8:44 PM
Unknown Object (File)
Fri, Mar 15, 8:43 PM
Unknown Object (File)
Tue, Mar 12, 1:12 AM
Unknown Object (File)
Fri, Mar 8, 4:12 AM
Unknown Object (File)
Feb 12 2024, 11:27 PM
Unknown Object (File)
Jan 24 2024, 5:17 PM
Unknown Object (File)
Dec 26 2023, 2:24 AM
Subscribers
None

Details

Summary

Rather than int flags we ++, use booleans for all command line args.
No functional change intended.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38441
Build 35330: arc lint + arc unit

Event Timeline

imp requested review of this revision.Apr 7 2021, 5:59 AM
imp created this revision.
usr.sbin/efivar/efivar.c
388

I guess expressions like this might be why int was used? I think this should still work with bools though.

415

These are still using arithmetic expressions.

Address markj's comments. Cast the bools to ints to add them, though this 'works' today.

markj added inline comments.
usr.sbin/efivar/efivar.c
409
This revision is now accepted and ready to land.Apr 9 2021, 1:53 PM
This revision was automatically updated to reflect the committed changes.