diff options
author | robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-06-04 13:22:14 +0000 |
---|---|---|
committer | robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-06-04 13:22:14 +0000 |
commit | 07ef911f18e30566d8a9d790e0bd69a836fd9d24 (patch) | |
tree | 352284a6918b54d51d554a1b8f39e7eb5b0644df /samplecode | |
parent | 009103d33008c4ee733b8a9fe09bb4c2c374a305 (diff) |
Fixed Linux compiler complaints
http://codereview.appspot.com/6269045/
git-svn-id: http://skia.googlecode.com/svn/trunk@4135 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'samplecode')
-rw-r--r-- | samplecode/SampleApp.cpp | 4 | ||||
-rw-r--r-- | samplecode/SampleUnitMapper.cpp | 21 |
2 files changed, 18 insertions, 7 deletions
diff --git a/samplecode/SampleApp.cpp b/samplecode/SampleApp.cpp index bc1b929301..fd19e89a5c 100644 --- a/samplecode/SampleApp.cpp +++ b/samplecode/SampleApp.cpp @@ -1051,7 +1051,9 @@ void SampleWindow::magnify(SkCanvas* canvas) { int count = canvas->save(); SkMatrix m = canvas->getTotalMatrix(); - m.invert(&m); + if (!m.invert(&m)) { + return; + } SkPoint offset, center; SkScalar mouseX = fMouseX * SK_Scalar1; SkScalar mouseY = fMouseY * SK_Scalar1; diff --git a/samplecode/SampleUnitMapper.cpp b/samplecode/SampleUnitMapper.cpp index d323ecb26a..09d0f14436 100644 --- a/samplecode/SampleUnitMapper.cpp +++ b/samplecode/SampleUnitMapper.cpp @@ -116,12 +116,17 @@ protected: canvas->drawPoints(SkCanvas::kPoints_PointMode, 2, &fPts[1], paint); } - SkPoint invertPt(SkScalar x, SkScalar y) { - SkPoint pt; + bool invertPt(SkScalar x, SkScalar y, SkPoint* result) { + if (NULL == result) + return true; + SkMatrix m; - fMatrix.invert(&m); - m.mapXY(x, y, &pt); - return pt; + if (!fMatrix.invert(&m)) { + return false; + } + + m.mapXY(x, y, result); + return true; } int hittest(SkScalar x, SkScalar y) { @@ -143,7 +148,11 @@ protected: virtual bool onClick(Click* click) { if (fDragIndex >= 0) { - fPts[fDragIndex] = invertPt(click->fCurr.fX, click->fCurr.fY); + if (!invertPt(click->fCurr.fX, click->fCurr.fY, + &fPts[fDragIndex])) { + return false; + } + this->setViews(); this->inval(NULL); return true; |