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

Issue 7386055: Fixed sk_getMetrics_utf32_xy function (Closed)

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

Description

Fixed sk_getMetrics_utf32_xy function The unused parameters x and y here seems to be a mistake. Committed: https://code.google.com/p/skia/source/detail?r=7873

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M src/core/SkPaint.cpp View 2 chunks +2 lines, -2 lines 3 comments Download

Messages

Total messages: 2
sugoi
I think these unused parameters were a mistake. You tell me. https://codereview.appspot.com/7386055/diff/1/src/core/SkPaint.cpp File src/core/SkPaint.cpp (right): ...
11 years, 9 months ago (2013-02-26 20:34:51 UTC) #1
reed1
11 years, 9 months ago (2013-02-26 21:54:08 UTC) #2
lgtm

https://codereview.appspot.com/7386055/diff/1/src/core/SkPaint.cpp
File src/core/SkPaint.cpp (right):

https://codereview.appspot.com/7386055/diff/1/src/core/SkPaint.cpp#newcode845
src/core/SkPaint.cpp:845: return cache->getUnicharMetrics(uni, x, y);
good catch

https://codereview.appspot.com/7386055/diff/1/src/core/SkPaint.cpp#newcode1692
src/core/SkPaint.cpp:1692: void SkScalerContext::PostMakeRec(const SkPaint&,
SkScalerContext::Rec* rec) {
agreed, renaming a class type with the same word usually adds no value.
Sign in to reply to this message.

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