Changeset 73294 in webkit

Timestamp:

Dec 3, 2010 1:38:16 PM (13 years ago)

Author:

crogers@google.com

Message:

2010-12-03 Chris Rogers <​crogers@google.com>

Unreviewed build fix.

Fix M_E compile error on Chromium Win
​https://bugs.webkit.org/show_bug.cgi?id=50479

• platform/audio/AudioUtilities.cpp: (WebCore::AudioUtilities::discreteTimeConstantForSampleRate):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/audio/AudioUtilities.cpp (modified) (3 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-12-03  Chris Rogers  <crogers@google.com>
+
+        Unreviewed build fix.
+
+        Fix M_E compile error on Chromium Win
+        https://bugs.webkit.org/show_bug.cgi?id=50479
+
+        * platform/audio/AudioUtilities.cpp:
+        (WebCore::AudioUtilities::discreteTimeConstantForSampleRate):
+
 2010-12-03  Chris Rogers  <crogers@google.com>
 

trunk/WebCore/platform/audio/AudioUtilities.cpp

@@ -25 +25 @@
 #include "config.h"
 
+#if ENABLE(WEB_AUDIO)
+
 #include "AudioUtilities.h"
-
-#include <math.h>
+#include <wtf/MathExtras.h>
 
 namespace WebCore {
@@ -51 +52 @@
 double discreteTimeConstantForSampleRate(double timeConstant, double sampleRate)
 {
-    return 1.0 - pow(1.0 / M_E, 1.0 / (sampleRate * timeConstant));
+    // hardcoded value is temporary build fix for Windows.
+    // FIXME: replace hardcode 2.718282 with M_E until the correct MathExtras.h solution is determined.
+    return 1.0 - pow(1.0 / 2.718282, 1.0 / (sampleRate * timeConstant));
 }
     
@@ -57 +60 @@
 
 } // WebCore
+
+#endif // ENABLE(WEB_AUDIO)