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

Issue 273643007: Convert all SkRecordPattern matchers into SkRecord mutators. (Closed)

Created:
6 years, 7 months ago by mtklein_C
Modified:
6 years, 7 months ago
CC:
skia-review_googlegroups.com
Base URL:
https://skia.googlesource.com/skia.git@master
Visibility:
Public.

Description

Convert all SkRecordPattern matchers into SkRecord mutators. - Allow any return type from SkRecord mutators and visitors; - update existing calls to mutate and visit; - convert match to operator() in SkRecordPattern; - tidy up a few inelegant bits of old code in tests. The net result is that the generated code is much clearer. All the mutate() calls inline as you'd hope, and you can now actually follow along with the disassembly. BUG=skia:2378 Committed: http://code.google.com/p/skia/source/detail?r=14631

Patch Set 1 #

Patch Set 2 : unused #

Patch Set 3 : defaults #

Patch Set 4 : indent #

Patch Set 5 : dumper too #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -58 lines) Patch
M src/record/SkRecord.h View 1 2 3 4 chunks +19 lines, -14 lines 0 comments Download
M src/record/SkRecordDraw.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/record/SkRecordOpts.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/record/SkRecordPattern.h View 1 8 chunks +12 lines, -22 lines 0 comments Download
M tests/RecordOptsTest.cpp View 1 chunk +8 lines, -9 lines 0 comments Download
M tests/RecordTest.cpp View 1 chunk +10 lines, -9 lines 0 comments Download
M tests/RecorderTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tools/dump_record.cpp View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (0 generated)
mtklein
6 years, 7 months ago (2014-05-07 19:59:32 UTC) #1
bungeman-skia
Other than not magically having decltype, lgtm.
6 years, 7 months ago (2014-05-07 20:41:14 UTC) #2
mtklein
On 2014/05/07 20:41:14, bungeman1 wrote: > Other than not magically having decltype, lgtm. SOON.
6 years, 7 months ago (2014-05-07 20:42:47 UTC) #3
mtklein
The CQ bit was checked by mtklein@google.com
6 years, 7 months ago (2014-05-07 20:42:51 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/273643007/20001
6 years, 7 months ago (2014-05-07 20:43:33 UTC) #5
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). Please consider checking whether the failures are ...
6 years, 7 months ago (2014-05-07 20:50:34 UTC) #6
mtklein
The CQ bit was checked by mtklein@google.com
6 years, 7 months ago (2014-05-07 20:54:29 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/273643007/60001
6 years, 7 months ago (2014-05-07 20:55:02 UTC) #8
mtklein
The CQ bit was unchecked by mtklein@google.com
6 years, 7 months ago (2014-05-07 20:59:13 UTC) #9
mtklein
The CQ bit was checked by mtklein@google.com
6 years, 7 months ago (2014-05-07 21:00:03 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/273643007/60009
6 years, 7 months ago (2014-05-07 21:00:38 UTC) #11
commit-bot: I haz the power
6 years, 7 months ago (2014-05-07 21:16:23 UTC) #12
Message was sent while issue was closed.
Change committed as 14631

Powered by Google App Engine
This is Rietveld 408576698