Removing warning suppression flags in common_audio/.

Bug: webrtc:9251 Change-Id: I9cae182ceb5e6bd3d6a34dc1a336ee3900f4cc98 Reviewed-on: https://webrtc-review.googlesource.com/86946 Reviewed-by: Karl Wiberg <[email protected]> Commit-Queue: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/master@{#23839}
darkGoldC · Jul 4, 2018 · f9c2952 · f9c2952
1 parent e12c1fe
commit f9c2952
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/common_audio/BUILD.gn b/common_audio/BUILD.gn
@@ -81,10 +81,6 @@ rtc_static_library("common_audio") {
     deps += [ ":common_audio_neon" ]
   }
 
-  if (is_win) {
-    cflags = [ "/wd4334" ]  # Ignore warning on shift operator promotion.
-  }
-
   public_configs = [ ":common_audio_config" ]
 
   if (!build_with_chromium && is_clang) {
@@ -207,10 +203,6 @@ rtc_source_set("common_audio_c") {
     ]
   }
 
-  if (is_win) {
-    cflags = [ "/wd4334" ]  # Ignore warning on shift operator promotion.
-  }
-
   public_configs = [ ":common_audio_config" ]
   deps = [
     ":common_audio_c_arm_asm",

diff --git a/common_audio/real_fourier.cc b/common_audio/real_fourier.cc
@@ -31,7 +31,7 @@ int RealFourier::FftOrder(size_t length) {
 
 size_t RealFourier::FftLength(int order) {
   RTC_CHECK_GE(order, 0);
-  return static_cast<size_t>(1 << order);
+  return size_t{1} << order;
 }
 
 size_t RealFourier::ComplexLength(int order) {

diff --git a/common_audio/signal_processing/complex_fft.c b/common_audio/signal_processing/complex_fft.c
@@ -166,7 +166,7 @@ int WebRtcSpl_ComplexIFFT(int16_t frfi[], int stages, int mode)
     /* The 1024-value is a constant given from the size of kSinTable1024[],
      * and should not be changed depending on the input parameter 'stages'
      */
-    n = 1 << stages;
+    n = ((size_t)1) << stages;
     if (n > 1024)
         return -1;