Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(281)

Issue 9411001: clang: -Wswitch warning tweaks. (Closed)

Created:
8 years, 10 months ago by Nico
Modified:
8 years, 10 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews, Ryan Sleevi
Visibility:
Public.

Description

clang: -Wswitch warning tweaks. yasm: Switch/enum warnings moved from -Wswitch-enum to -Wswitch in clang, rename flag. Roll ffmpeg 120865:122092 Only change: -Wno-switch-enum -> -Wno-switch Roll ICU 121777:122103 Only change: Add -Wno-switch to ICU (two CLs). Roll NSS 117974:122093 r119755: Allow projects dependent on NSS to specify whether to build the complete NSS. r122093: -Wno-switch-enum -> -Wno-switch BUG=102390 TEST=fewer warnings when building TBR=evan Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=122106

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : icu 122103 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -5 lines) Patch
M DEPS View 1 2 3 chunks +3 lines, -3 lines 0 comments Download
M third_party/yasm/yasm.gyp View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Nico
(rsleevi: y u no roll deps after change?)
8 years, 10 months ago (2012-02-15 16:51:29 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/9411001/1
8 years, 10 months ago (2012-02-15 16:53:09 UTC) #2
commit-bot: I haz the power
Can't apply patch for file DEPS. While running patch -p1 --forward --force; patching file DEPS ...
8 years, 10 months ago (2012-02-15 16:53:12 UTC) #3
commit-bot: I haz the power
8 years, 10 months ago (2012-02-15 16:58:53 UTC) #4

Powered by Google App Engine
This is Rietveld 408576698