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

Issue 69033005: Don't warn on missing prop synthesis in non-chromium code for iOS. (Closed)

Created:
7 years, 1 month ago by justincohen
Modified:
7 years, 1 month ago
CC:
chromium-reviews
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Don't warn on missing prop synthesis in non-chromium code for iOS. Certain internal directories fire many many warnings for -Wobjc-missing-property-synthesis. BUG=None TEST=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=234543

Patch Set 1 #

Patch Set 2 : Address Nico Comments #

Total comments: 2

Patch Set 3 : Rebase + Nico comment #

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

Messages

Total messages: 9 (0 generated)
justincohen
PTAl!
7 years, 1 month ago (2013-11-12 00:14:54 UTC) #1
justincohen
7 years, 1 month ago (2013-11-12 00:16:55 UTC) #2
Nico
As always, please answer "why?" in the CL description. Especially for the plugin part, which ...
7 years, 1 month ago (2013-11-12 00:25:08 UTC) #3
justincohen
I reduced the CL to just 'CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTHESIS': 'NO', PTAL!
7 years, 1 month ago (2013-11-12 02:46:34 UTC) #4
Nico
lgtm https://codereview.chromium.org/69033005/diff/60001/build/common.gypi File build/common.gypi (right): https://codereview.chromium.org/69033005/diff/60001/build/common.gypi#newcode2488 build/common.gypi:2488: 'CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTHESIS': 'NO', Maybe add a comment above this ...
7 years, 1 month ago (2013-11-12 04:29:27 UTC) #5
justincohen
https://codereview.chromium.org/69033005/diff/60001/build/common.gypi File build/common.gypi (right): https://codereview.chromium.org/69033005/diff/60001/build/common.gypi#newcode2488 build/common.gypi:2488: 'CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTHESIS': 'NO', On 2013/11/12 04:29:27, Nico wrote: > Maybe ...
7 years, 1 month ago (2013-11-12 12:44:46 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/justincohen@chromium.org/69033005/110001
7 years, 1 month ago (2013-11-12 12:49:08 UTC) #7
stuartmorgan1
lgtm
7 years, 1 month ago (2013-11-12 15:31:43 UTC) #8
commit-bot: I haz the power
7 years, 1 month ago (2013-11-12 17:01:21 UTC) #9
Message was sent while issue was closed.
Change committed as 234543

Powered by Google App Engine
This is Rietveld 408576698