Bug 879651 - Choose between integers or float based on the platform rather than the CPU architecture. r=blassey, a=akeybl
authorPaul Adenot <paul@paul.cx>
Tue, 11 Jun 2013 02:07:22 +0200
changeset 137771 47ee3835df837fa55379134ba738a0a170ac6fce
parent 137770 09b627ea659b4327a3723f5b9b58a638b2486385
child 137772 50aa2dd3b0bd41197c1d9aa0ba400357623ed604
push id2554
push userryanvm@gmail.com
push dateTue, 11 Jun 2013 13:21:12 +0000
treeherdermozilla-beta@47ee3835df83 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersblassey, akeybl
bugs879651
milestone22.0
Bug 879651 - Choose between integers or float based on the platform rather than the CPU architecture. r=blassey, a=akeybl
configure.in
--- a/configure.in
+++ b/configure.in
@@ -5348,28 +5348,27 @@ AC_SUBST(MOZ_WEBRTC)
 AC_SUBST(MOZ_WEBRTC_LEAKING_TESTS)
 AC_SUBST(MOZ_WEBRTC_SIGNALING)
 AC_SUBST(MOZ_PEERCONNECTION)
 AC_SUBST(MOZ_WEBRTC_IN_LIBXUL)
 AC_SUBST(MOZ_WEBRTC_ASSERT_ALWAYS)
 AC_SUBST(MOZ_SCTP)
 AC_SUBST(MOZ_SRTP)
 
-case "$target_cpu" in
-arm*)
+dnl Use integers over floats for audio on B2G and Android, because audio
+dnl backends for those platforms don't support floats.
+if test "$OS_TARGET" = "Android"; then
     MOZ_SAMPLE_TYPE_S16=1
     AC_DEFINE(MOZ_SAMPLE_TYPE_S16)
     AC_SUBST(MOZ_SAMPLE_TYPE_S16)
-;;
-*)
+else
     MOZ_SAMPLE_TYPE_FLOAT32=1
     AC_DEFINE(MOZ_SAMPLE_TYPE_FLOAT32)
     AC_SUBST(MOZ_SAMPLE_TYPE_FLOAT32)
-;;
-esac
+fi
 
 dnl ========================================================
 dnl = Disable Speech API code
 dnl ========================================================
 MOZ_ARG_DISABLE_BOOL(webspeech,
 [  --disable-webspeech        Disable support for HTML Speech API],
     MOZ_WEBSPEECH=,
     MOZ_WEBSPEECH=1)