diff options
author | bjornv@webrtc.org <bjornv@webrtc.org> | 2014-09-29 09:40:38 +0000 |
---|---|---|
committer | bjornv@webrtc.org <bjornv@webrtc.org> | 2014-09-29 09:40:38 +0000 |
commit | ff91ab88424b7b9f1f9fe9abddbe78c188585a3d (patch) | |
tree | 6517f3c70d0f73c72316362a580696a73014f69e /common_audio/signal_processing/signal_processing_unittest.cc | |
parent | a30618fcaefc73812ed49cfa94232ca73f0c76c7 (diff) | |
download | webrtc-ff91ab88424b7b9f1f9fe9abddbe78c188585a3d.tar.gz |
common_audio refactoring: Removed macro WEBRTC_SPL_LSHIFT_U32
The macro is a trivial shift operator including a cast before shift. There is no guard against negative shifts. Replaced with << at place and added casts when necessary.
Affects both fixed and float point versions of iSAC
BUG=3348,3353
TESTED=locally on linux and trybots
R=kwiberg@webrtc.org, tina.legrand@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/27369004
git-svn-id: http://webrtc.googlecode.com/svn/trunk/webrtc@7320 4adac7df-926f-26a2-2b94-8c16560cd09d
Diffstat (limited to 'common_audio/signal_processing/signal_processing_unittest.cc')
-rw-r--r-- | common_audio/signal_processing/signal_processing_unittest.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/common_audio/signal_processing/signal_processing_unittest.cc b/common_audio/signal_processing/signal_processing_unittest.cc index 3fa3d2ee..d5cc5f07 100644 --- a/common_audio/signal_processing/signal_processing_unittest.cc +++ b/common_audio/signal_processing/signal_processing_unittest.cc @@ -72,7 +72,6 @@ TEST_F(SplTest, MacroTest) { EXPECT_EQ(32766, WEBRTC_SPL_LSHIFT_W32(a, 1)); EXPECT_EQ(8191u, WEBRTC_SPL_RSHIFT_U32(a, 1)); - EXPECT_EQ(32766u, WEBRTC_SPL_LSHIFT_U32(a, 1)); EXPECT_EQ(1470, WEBRTC_SPL_RAND(A)); |