1152016 - Suppress MJPEG fprintf() warnings in libyuv

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, defect, P1)

Description

[Randell Jesup [:jesup] (needinfo me)]

jpeg-turbo by default dumps warnings to stderr (including warnings about extraneous bytes, which happen All The Time in MJPEG data).

We really don't want to dump this all to stderr by default

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: suppress fprintf(stderr)'s from jpeg in MJPEG decode

Pretty trivial patch to leverage the ability to redefine warning output for jpeg

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/483da6a62ae4

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/483da6a62ae4

Comment 4

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8589259 [details] [diff] [review]
suppress fprintf(stderr)'s from jpeg in MJPEG decode

Approval Request Comment
See bug 1151628's request

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8589259 [details] [diff] [review]
suppress fprintf(stderr)'s from jpeg in MJPEG decode

I'm approving this prereq for bug 1151628 for Aurora. As we're up to Beta 7, I'm going to leave the call for Beta up to Sylvestre.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d3af56f0df91

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8589259 [details] [diff] [review]
suppress fprintf(stderr)'s from jpeg in MJPEG decode

[Triage Comment]
Let's take it as it is needed for bug 1151628

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/97d33db56113

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/97d33db56113