Differential D19051

else
ClosedPublic
Actions

Authored by freqlabs on Feb 1 2019, 2:49 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Feb 8, 12:38 PM

	Unknown Object (File)
	Sun, Feb 8, 12:03 AM

	Unknown Object (File)
	Mon, Feb 2, 11:49 AM

	Unknown Object (File)
	Sat, Jan 31, 2:14 PM

	Unknown Object (File)
	Jan 9 2026, 12:45 PM

	Unknown Object (File)
	Dec 27 2025, 6:55 PM

	Unknown Object (File)
	Dec 24 2025, 12:31 PM

	Unknown Object (File)
	Nov 20 2025, 10:39 PM

View All 88 Files

Subscribers

imp

Details

Reviewers

mav

Commits

rS343727: Use switch instead of chained if/else to improve readability.

Summary

Using a switch statement helps improve readability.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

freqlabs created this revision.Feb 1 2019, 2:49 AM

Herald added a subscriber: imp. · View Herald TranscriptFeb 1 2019, 2:49 AM

Looks good to me.

This revision is now accepted and ready to land.Feb 1 2019, 3:50 AM

Closed by commit rS343727: Use switch instead of chained if/else to improve readability. (authored by mav). · Explain WhyFeb 4 2019, 1:21 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

cam/

scsi/

scsi_enc_ses.c

22 lines

Diff 53569

View Options

head/sys/cam/scsi/scsi_enc_ses.c

cam/scsi: Use switch instead of chained if/elseClosedPublicActions