diff options
| author | bjornv@webrtc.org <bjornv@webrtc.org> | 2014-08-25 11:42:42 +0000 |
|---|---|---|
| committer | bjornv@webrtc.org <bjornv@webrtc.org> | 2014-08-25 11:42:42 +0000 |
| commit | 9cfe80329340de096ed8e7e2d8b109611d85b3d0 (patch) | |
| tree | 07def622013dac64ce8f77b341e9b0f88e878eab /common_audio | |
| parent | 62772e9d8c7b2992ed981e3505726b119c0fe356 (diff) | |
| download | webrtc-9cfe80329340de096ed8e7e2d8b109611d85b3d0.tar.gz | |
Refactoring common_audio: Replace trivial multiplication macro
This multiplication macro literally use the '*' operator, so there is no need for it.
BUG=3348,3353
TESTED=locally on linux and trybots
R=kwiberg@webrtc.org, tina.legrand@webrtc.org, turaj@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/22109004
git-svn-id: http://webrtc.googlecode.com/svn/trunk/webrtc@6964 4adac7df-926f-26a2-2b94-8c16560cd09d
Diffstat (limited to 'common_audio')
| -rw-r--r-- | common_audio/signal_processing/include/signal_processing_library.h | 1 | ||||
| -rw-r--r-- | common_audio/signal_processing/signal_processing_unittest.cc | 1 |
2 files changed, 0 insertions, 2 deletions
diff --git a/common_audio/signal_processing/include/signal_processing_library.h b/common_audio/signal_processing/include/signal_processing_library.h index 2ca7aff2..8e1059e5 100644 --- a/common_audio/signal_processing/include/signal_processing_library.h +++ b/common_audio/signal_processing/include/signal_processing_library.h @@ -85,7 +85,6 @@ (C + (B >> 16) * A + (((uint32_t)(0x0000FFFF & B) * A) >> 16)) #define WEBRTC_SPL_SAT(a, b, c) (b > a ? a : b < c ? c : b) -#define WEBRTC_SPL_MUL_32_16(a, b) ((a) * (b)) // Shifting with negative numbers allowed // Positive means left shift diff --git a/common_audio/signal_processing/signal_processing_unittest.cc b/common_audio/signal_processing/signal_processing_unittest.cc index ddcf70a0..88ede79b 100644 --- a/common_audio/signal_processing/signal_processing_unittest.cc +++ b/common_audio/signal_processing/signal_processing_unittest.cc @@ -60,7 +60,6 @@ TEST_F(SplTest, MacroTest) { EXPECT_EQ(21, WEBRTC_SPL_SAT(a, A, B)); EXPECT_EQ(21, WEBRTC_SPL_SAT(a, B, A)); - EXPECT_EQ(-49149, WEBRTC_SPL_MUL_32_16(a, b)); // Shifting with negative numbers allowed int shift_amount = 1; // Workaround compiler warning using variable here. |