bsdinstall: replace multiple ifs with case
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F170648261: D28417.id.diff
	Sat, Sep 5, 7:34 PM

	F170644403: D28417.diff
	Sat, Sep 5, 7:05 PM

	F170566551: D28417.diff
	Sat, Sep 5, 10:52 AM

	Unknown Object (File)
	Fri, Sep 4, 9:20 AM

	Unknown Object (File)
	Fri, Sep 4, 1:58 AM

	Unknown Object (File)
	Thu, Sep 3, 6:48 PM

	Unknown Object (File)
	Thu, Sep 3, 12:39 PM

	Unknown Object (File)
	Wed, Sep 2, 3:24 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