Commit 7a8a9709 authored by tony@chromium.org's avatar tony@chromium.org

Cleanup of switch statements with default cases

https://bugs.webkit.org/show_bug.cgi?id=67808

Reviewed by Adam Barth.

No new tests, just a small refactoring.

* page/WebKitAnimation.cpp:
(WebCore::WebKitAnimation::fillMode):
* platform/audio/Distance.cpp:
(WebCore::DistanceEffect::gain):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@94823 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 0c65fb0f
2011-09-08 Tony Chang <tony@chromium.org>
Cleanup of switch statements with default cases
https://bugs.webkit.org/show_bug.cgi?id=67808
Reviewed by Adam Barth.
No new tests, just a small refactoring.
* page/WebKitAnimation.cpp:
(WebCore::WebKitAnimation::fillMode):
* platform/audio/Distance.cpp:
(WebCore::DistanceEffect::gain):
2011-09-08 Eric Seidel <eric@webkit.org>
[BiDi] Add support for the BDI element
......@@ -94,17 +94,15 @@ WebKitAnimation::FillMode WebKitAnimation::fillMode() const
switch (m_keyframeAnimation->animation()->fillMode()) {
case AnimationFillModeNone:
return FILL_NONE;
break;
case AnimationFillModeForwards:
return FILL_FORWARDS;
break;
case AnimationFillModeBackwards:
return FILL_BACKWARDS;
break;
default:
case AnimationFillModeBoth:
return FILL_BOTH;
break;
}
ASSERT_NOT_REACHED();
return FILL_BOTH;
}
void WebKitAnimation::pause()
......
......@@ -60,17 +60,13 @@ double DistanceEffect::gain(double distance)
switch (m_model) {
case ModelLinear:
return linearGain(distance);
break;
case ModelInverse:
return inverseGain(distance);
break;
case ModelExponential:
return exponentialGain(distance);
break;
default:
return 0.0;
}
ASSERT_NOT_REACHED();
return 0.0;
}
double DistanceEffect::linearGain(double distance)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment