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

Issue 85763002: remove unused reduce order code (Closed)

Created:
7 years ago by caryclark
Modified:
7 years ago
Reviewers:
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

remove unused reduce order code BUG= Committed: https://code.google.com/p/skia/source/detail?r=12377

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+64 lines, -236 lines) Patch
M src/pathops/SkDCubicIntersection.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/pathops/SkDCubicToQuads.cpp View 2 chunks +3 lines, -5 lines 0 comments Download
M src/pathops/SkOpSegment.h View 4 chunks +13 lines, -9 lines 0 comments Download
M src/pathops/SkReduceOrder.h View 1 chunk +2 lines, -6 lines 0 comments Download
M src/pathops/SkReduceOrder.cpp View 13 chunks +16 lines, -183 lines 0 comments Download
M tests/PathOpsCubicIntersectionTest.cpp View 1 chunk +2 lines, -4 lines 0 comments Download
M tests/PathOpsCubicLineIntersectionTest.cpp View 1 chunk +1 line, -2 lines 0 comments Download
M tests/PathOpsCubicQuadIntersectionTest.cpp View 1 chunk +2 lines, -3 lines 0 comments Download
M tests/PathOpsCubicReduceOrderTest.cpp View 9 chunks +17 lines, -16 lines 0 comments Download
M tests/PathOpsQuadIntersectionTest.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M tests/PathOpsQuadLineIntersectionTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tests/PathOpsQuadLineIntersectionThreadedTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tests/PathOpsQuadReduceOrderTest.cpp View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
commit-bot: I haz the power
No comments yet.
7 years ago (2013-11-25 13:30:11 UTC) #1
caryclark
lgtm
7 years ago (2013-11-25 13:32:28 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
7 years ago (2013-11-25 13:32:51 UTC) #3
caryclark
7 years ago (2013-11-25 14:19:55 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r12377 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698