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

Issue 10097002: clang: Remove -Wno-c++11-compat. (Closed)

Created:
8 years, 8 months ago by Nico
Modified:
8 years, 8 months ago
Reviewers:
hans
CC:
chromium-reviews
Visibility:
Public.

Description

clang: Remove -Wno-c++11-compat. It has been removed from -Wall in clang r143248 (Oct 2011), so there's no need to explicitly disable it. No behavior change. BUG=none TEST=none TBR=hans NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=132367

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -6 lines) Patch
M build/common.gypi View 2 chunks +0 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Nico
Probably worth to try and turn this flag _on_. It looks like WTF/nullptr.h WebKit doesn't ...
8 years, 8 months ago (2012-04-15 19:25:02 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/10097002/1
8 years, 8 months ago (2012-04-15 19:25:13 UTC) #2
commit-bot: I haz the power
Try job failure for 10097002-1 (retry) on win_rel for step "browser_tests". It's a second try, ...
8 years, 8 months ago (2012-04-15 21:51:35 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/10097002/1
8 years, 8 months ago (2012-04-15 21:55:36 UTC) #4
commit-bot: I haz the power
Change committed as 132367
8 years, 8 months ago (2012-04-15 21:55:43 UTC) #5
hans
8 years, 8 months ago (2012-04-16 16:46:59 UTC) #6
On 2012/04/15 21:55:43, I haz the power (commit-bot) wrote:
> Change committed as 132367

lgtm

Powered by Google App Engine
This is Rietveld 408576698