Remove unnecessary default from a switch statement.
Don't check for impossible enum element value, it's better to not have the default clause at all to get a warning from g++ if this enum ever changes later. And for now the presence of default results in a warning from Coverity about unreachable code. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@73147 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
@@ -2880,10 +2880,6 @@ wxImage::HSVValue wxImage::RGBtoHSV(const RGBValue& rgb)
|
|||||||
case BLUE:
|
case BLUE:
|
||||||
hue = 4.0 + (red - green) / deltaRGB;
|
hue = 4.0 + (red - green) / deltaRGB;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
default:
|
|
||||||
wxFAIL_MSG(wxT("hue not specified"));
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
hue /= 6.0;
|
hue /= 6.0;
|
||||||
|
Reference in New Issue
Block a user