Project

General

Profile

Bug #5668

Switch statement with single case in vdev_disk.c seems inappropriate

Added by Sam Zaydel over 5 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
zfs - Zettabyte File System
Start date:
2015-02-27
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage
Gerrit CR:

Description

It looks to me like this is something that should be changed to an if statement, since there is really only one case and the default effectively matches anything other than the case does. Seems like an if clause is more suitable here.

https://github.com/illumos/illumos-gate/blob/master/usr/src/uts/common/fs/zfs/vdev_disk.c#L746

No data to display

Also available in: Atom PDF