bsdinstall: replace multiple ifs with case
ClosedPublic
Actions

Authored by emaste on Jan 29 2021, 7:16 PM.

Tags

None

Referenced Files

	F140925870: D28417.id.diff
	Mon, Dec 29, 5:32 PM

	F140901212: D28417.id83118.diff
	Mon, Dec 29, 10:22 AM

	Unknown Object (File)
	Mon, Dec 22, 12:02 AM

	Unknown Object (File)
	Wed, Dec 17, 8:03 PM

	Unknown Object (File)
	Wed, Dec 17, 2:22 PM

	Unknown Object (File)
	Sun, Dec 14, 6:06 AM

	Unknown Object (File)
	Wed, Dec 10, 6:17 PM

	Unknown Object (File)
	Fri, Dec 5, 3:53 AM

View All Files

Subscribers

None

Details

Reviewers

allanjude
robak
dteske

Commits

rG8ea77f39ec17: bsdinstall: replace multiple ifs with case
rGfbc57e2df95b: bsdinstall: replace multiple ifs with case

Summary

Reduce copy-paste and use a more typical construct.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste requested review of this revision.Jan 29 2021, 7:16 PM

emaste created this revision.

emaste added a child revision: D28418: bsdinstall: add knob to set ASLR sysctls.Jan 29 2021, 7:19 PM

with context

emaste added a reviewer: dteske.Feb 5 2021, 5:40 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 20 2021, 4:57 PM

Closed by commit rGfbc57e2df95b: bsdinstall: replace multiple ifs with case (authored by emaste). · Explain Why

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rGfbc57e2df95b: bsdinstall: replace multiple ifs with case.

emaste added a commit: rG8ea77f39ec17: bsdinstall: replace multiple ifs with case.Mar 5 2021, 5:53 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bsdinstall/

scripts/

hardening

46 lines

Diff 83431

View Options

usr.sbin/bsdinstall/scripts/hardening

bsdinstall: replace multiple ifs with caseClosedPublicActions