828713 - soundtouch_config.h checks the non-existing MOZ_SAMPLE_TYPE_S16LE symbol

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[(no longer active)]

See <http://mxr.mozilla.org/mozilla-central/source/media/libsoundtouch/src/soundtouch_config.h#16>.  Does this mean to check for MOZ_SAMPLE_TYPE_S16 instead?

(Note that this could be a left-over from bug 783953.)

Comment 1

[Paul Adenot (:padenot)]

Attachment: soundtouch_config.h checks the non-existing MOZ_SAMPLE_TYPE_S16LE symbol. r=

Oops.

In fact, I believe it does not matter too much, since we don't use integers
for audio on desktop, the only place where we could have MMX. But we just made
fennec work on mobile x86, that may have MMX, but I'm not sure what we should do
regarding floating point/integer for audio there.

Good catch :-).

Comment 2

[(no longer active)]

On a related note, seems like we do require FPU for Fennec on Arm-v6 for now...

Comment 3

[Paul Adenot (:padenot)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e9c8447fb197

Comment 4

[Ed Morley [:emorley]]

Push backed out for Windows pgo-only mochitest-1 timeouts in media tests, since the backout of just 1abf4c88f8f1 didn't work (see bug 793274 comment 12 for example logs):
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2912b7e727a

Comment 5

[Paul Adenot (:padenot)]

After bisecting the queue on try, this patch has been proved to cause no problem [0], note the patch at the top of the queue that requests a PGO build. The problem remains in the other part of the queue [1].

This has been pushed as https://hg.mozilla.org/integration/mozilla-inbound/rev/6ba8350ad2a6

[0]: https://tbpl.mozilla.org/?tree=Try&rev=516ddf4c963c
[1]: https://tbpl.mozilla.org/?tree=Try&rev=418c581b8023

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/6ba8350ad2a6