1708433 - Perma /builds/worker/checkouts/gecko/dom/media/mediasink/AudioDecoderInputTrack.cpp:216:14: error: unused variable 'rv' [-Werror,-Wunused-variable] when Gecko 90 merges to Beta on 2021-05-31

Status: VERIFIED FIXED

(Core :: Audio/Video: MediaStreamGraph, defect)

Description

[Iulian Moraru]

Central as beta simulation: https://treeherder.mozilla.org/jobs?repo=try&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=5a10c123be15fa9704b7df4a2647fae9f6df9c90&selectedTaskRun=Ccm8G7nJRvWLSTsNjxDXFg.0

How to run this simulation: https://wiki.mozilla.org/Sheriffing/How_To/Beta_simulations#TRUNK_AS_EARLY_BETA

Failure log: https://treeherder.mozilla.org/logviewer?job_id=338149607&repo=try&lineNumber=9947

[task 2021-04-29T12:24:25.617Z] 12:24:25 ERROR - /builds/worker/checkouts/gecko/dom/media/mediasink/AudioDecoderInputTrack.cpp:216:14: error: unused variable 'rv' [-Werror,-Wunused-variable]
[task 2021-04-29T12:24:25.617Z] 12:24:25 INFO - const bool rv = mSPSCQueue.Enqueue(data);
[task 2021-04-29T12:24:25.617Z] 12:24:25 INFO - ^

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1708433 - add 'unused' for the result used for assertion. r?padenot

Depends on D113822

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8d038e2b675f
add 'unused' for the result used for assertion. r=padenot

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/8d038e2b675f

Comment 4

[Alexandru Michis [:malexandru]]

Verified fixed in today's central-as-early-beta simulation.