turn off select warnings for ICU

RESOLVED FIXED in Firefox 55

Status

()

Core
Build Config
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

We don't really care about these warnings, and they're just cluttering
the build.
Created attachment 8868659 [details] [diff] [review]
turn off select warnings for ICU
Attachment #8868659 - Flags: review?(ted)
Attachment #8868659 - Flags: review?(ted) → review+

Comment 2

7 months ago
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6cb366239c87
turn off select warnings for ICU; r=ted.mielczarek

Comment 3

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6cb366239c87
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
This showed up on perfherder:

== Change summary for alert #6679 (as of May 17 2017 18:50 UTC) ==

Improvements:

  4%  compiler warnings summary linux32 debug      473.00 -> 456.17
  4%  compiler warnings summary linux64-stylo debug 480.00 -> 463.17
  4%  compiler warnings summary linux64 debug      479.00 -> 462.25
  3%  compiler warnings summary android-4-0-armv7-api15 debug 582.00 -> 565.17
  1%  compiler warnings summary linux32 opt        1,157.00 -> 1,140.17
  1%  compiler warnings summary linux64-stylo opt  1,165.92 -> 1,149.08
  1%  compiler warnings summary linux64 opt        1,165.00 -> 1,148.17
  1%  compiler warnings summary linux64 opt valgrind1,163.92 -> 1,147.33
  1%  compiler warnings summary android-api-15-gradle opt 1,219.92 -> 1,203.00
  1%  compiler warnings summary android-4-0-armv7-api15 opt 1,219.92 -> 1,203.08
  1%  compiler warnings summary linux64 opt static-analysis340.62 -> 336.50
  1%  compiler warnings summary linux64 debug static-analysis367.58 -> 363.42

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=6679
You need to log in before you can comment on or make changes to this bug.