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)
Thu, Dec 5, 7:41 PM
Unknown Object (File)
Sun, Nov 24, 3:17 PM
Unknown Object (File)
Nov 17 2024, 10:25 PM
Unknown Object (File)
Nov 5 2024, 5:56 PM
Unknown Object (File)
Oct 24 2024, 10:49 PM
Unknown Object (File)
Oct 22 2024, 6:54 PM
Unknown Object (File)
Oct 1 2024, 9:22 PM
Unknown Object (File)
Sep 30 2024, 10:39 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.