Project

General

Profile

Actions

Bug #5668

open

Switch statement with single case in vdev_disk.c seems inappropriate

Added by Sam Zaydel over 8 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:
External Bug:

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

Actions

Also available in: Atom PDF