Commit 4b921433 authored by Kuznetsov Alexey's avatar Kuznetsov Alexey Committed by Commit Bot

Replace PI constant from base

In chromium code audio_utility.cc we just need to use PI constant from
//base instead from local file.
This is some polishing.

Change-Id: I71d7042a9ed40c563c67bab2237b267c822a6961
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2027367Reviewed-by: default avatarPeter Kasting <pkasting@chromium.org>
Reviewed-by: default avatarYuri Wiitala <miu@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737230}
parent f1b0cfbf
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
#include <vector> #include <vector>
#include "base/logging.h" #include "base/logging.h"
#include "base/numerics/math_constants.h"
#include "base/time/time.h" #include "base/time/time.h"
#include "media/base/audio_bus.h" #include "media/base/audio_bus.h"
#include "media/cast/test/utility/audio_utility.h" #include "media/cast/test/utility/audio_utility.h"
...@@ -13,8 +14,6 @@ ...@@ -13,8 +14,6 @@
namespace media { namespace media {
namespace cast { namespace cast {
const double Pi = 3.14159265358979323846;
TestAudioBusFactory::TestAudioBusFactory(int num_channels, TestAudioBusFactory::TestAudioBusFactory(int num_channels,
int sample_rate, int sample_rate,
float sine_wave_frequency, float sine_wave_frequency,
...@@ -113,8 +112,8 @@ bool EncodeTimestamp(uint16_t timestamp, ...@@ -113,8 +112,8 @@ bool EncodeTimestamp(uint16_t timestamp,
for (size_t i = 0; i < length; i++) { for (size_t i = 0; i < length; i++) {
double mix_of_components = 0.0; double mix_of_components = 0.0;
for (size_t f = 0; f < frequencies.size(); f++) { for (size_t f = 0; f < frequencies.size(); f++) {
mix_of_components += sin((i + sample_offset) * Pi * 2.0 * frequencies[f] / mix_of_components += sin((i + sample_offset) * base::kPiDouble * 2.0 *
kSamplingFrequency); frequencies[f] / kSamplingFrequency);
} }
mix_of_components /= kNumBits + 1; mix_of_components /= kNumBits + 1;
DCHECK_LE(fabs(mix_of_components), 1.0); DCHECK_LE(fabs(mix_of_components), 1.0);
...@@ -136,8 +135,10 @@ double DecodeOneFrequency(const float* samples, ...@@ -136,8 +135,10 @@ double DecodeOneFrequency(const float* samples,
double sin_sum = 0.0; double sin_sum = 0.0;
double cos_sum = 0.0; double cos_sum = 0.0;
for (size_t i = 0; i < length; i++) { for (size_t i = 0; i < length; i++) {
sin_sum += samples[i] * sin(i * Pi * 2 * frequency / kSamplingFrequency); sin_sum += samples[i] *
cos_sum += samples[i] * cos(i * Pi * 2 * frequency / kSamplingFrequency); sin(i * base::kPiDouble * 2 * frequency / kSamplingFrequency);
cos_sum += samples[i] *
cos(i * base::kPiDouble * 2 * frequency / kSamplingFrequency);
} }
return sqrt(sin_sum * sin_sum + cos_sum * cos_sum); return sqrt(sin_sum * sin_sum + cos_sum * cos_sum);
} }
......
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