bsdinstall: replace multiple ifs with case
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F161351485: D28417.id.diff
	Fri, Jul 3, 2:04 AM

	F161347656: D28417.id84329.diff
	Fri, Jul 3, 1:15 AM

	F161346990: D28417.id83118.diff
	Fri, Jul 3, 1:07 AM

	F161336756: D28417.id83431.diff
	Thu, Jul 2, 10:51 PM

	F161274440: D28417.id83118.diff
	Thu, Jul 2, 8:41 AM

	Unknown Object (File)
	Wed, Jul 1, 3:59 AM

	Unknown Object (File)
	Wed, Jul 1, 2:06 AM

	Unknown Object (File)
	Tue, Jun 30, 11:00 PM

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

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 84329

View Options

usr.sbin/bsdinstall/scripts/hardening

bsdinstall: replace multiple ifs with caseClosedPublicActions