Differential D19051

else
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F170624874: D19051.id53569.diff
	Sat, Sep 5, 5:00 PM

	F170624824: D19051.id53569.diff
	Sat, Sep 5, 5:00 PM

	F170595769: D19051.diff
	Sat, Sep 5, 1:54 PM

	F170581676: D19051.id53501.diff
	Sat, Sep 5, 12:27 PM

	F170525435: D19051.id.diff
	Sat, Sep 5, 6:01 AM

	F170497373: D19051.id.diff
	Sat, Sep 5, 2:45 AM

	Unknown Object (File)
	Wed, Sep 2, 5:58 PM

	Unknown Object (File)
	Wed, Sep 2, 10:38 AM

View All 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