Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1177)

Issue 6597063: build iOS with 'make all' (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 8 months ago by caryclark1
Modified:
11 years, 8 months ago
Reviewers:
EricB, bsalomon
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

build iOS with 'make all' This builds all skia tests by treating iOS tools as executable applications. A few warnings were fixed as well. Removed old trace draw code and remnants. Committed: https://code.google.com/p/skia/source/detail?r=5776

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+205 lines, -41 lines) Patch
M bench/benchmain.cpp View 2 chunks +9 lines, -1 line 0 comments Download
M experimental/SimpleiOSApp/SimpleApp.mm View 4 chunks +14 lines, -1 line 0 comments Download
A experimental/SimpleiOSApp/tool-Info.plist View 1 chunk +43 lines, -0 lines 0 comments Download
M experimental/iOSSampleApp/Shared/skia_ios.mm View 1 chunk +4 lines, -0 lines 0 comments Download
M experimental/iOSSampleApp/SkSampleUIView.mm View 1 6 chunks +11 lines, -24 lines 0 comments Download
M gm/gmmain.cpp View 2 chunks +9 lines, -1 line 0 comments Download
M gyp/apptype_console.gypi View 1 chunk +49 lines, -0 lines 0 comments Download
M gyp/common_variables.gypi View 2 chunks +2 lines, -6 lines 0 comments Download
M samplecode/SampleApp.cpp View 1 1 chunk +5 lines, -0 lines 0 comments Download
M src/core/SkPaint.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M tests/skia_test.cpp View 2 chunks +9 lines, -1 line 0 comments Download
M tools/bench_pictures_main.cpp View 2 chunks +8 lines, -1 line 0 comments Download
M tools/pinspect.cpp View 2 chunks +8 lines, -1 line 0 comments Download
M tools/render_pictures_main.cpp View 2 chunks +8 lines, -1 line 0 comments Download
M tools/skdiff_main.cpp View 2 chunks +8 lines, -1 line 0 comments Download
M tools/skhello.cpp View 2 chunks +8 lines, -1 line 0 comments Download
M tools/skimage_main.cpp View 2 chunks +8 lines, -1 line 0 comments Download

Messages

Total messages: 5
caryclark1
11 years, 8 months ago (2012-10-02 17:08:49 UTC) #1
bsalomon
LGTM w/ one question. https://codereview.appspot.com/6597063/diff/1/samplecode/SampleApp.cpp File samplecode/SampleApp.cpp (right): https://codereview.appspot.com/6597063/diff/1/samplecode/SampleApp.cpp#newcode2461 samplecode/SampleApp.cpp:2461: bool gDoTraceDraw; What's this?
11 years, 8 months ago (2012-10-02 17:37:28 UTC) #2
caryclark1
https://codereview.appspot.com/6597063/diff/1/samplecode/SampleApp.cpp File samplecode/SampleApp.cpp (right): https://codereview.appspot.com/6597063/diff/1/samplecode/SampleApp.cpp#newcode2461 samplecode/SampleApp.cpp:2461: bool gDoTraceDraw; Obsolete code. Removed it and its prodgeny. ...
11 years, 8 months ago (2012-10-02 18:29:21 UTC) #3
bsalomon
LGTM
11 years, 8 months ago (2012-10-02 18:32:09 UTC) #4
EricB
11 years, 8 months ago (2012-10-02 18:32:53 UTC) #5
On 2012/10/02 18:32:09, bsalomon wrote:
> LGTM

LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b