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

Issue 6304098: fixing compiler warnings in SkAAClip (Closed)

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

Description

fixing compiler warnings in SkAAClip ../src/core/SkAAClip.cpp: In member function ‘bool SkAAClip::quickContains(int, int, int, int) const’: ../src/core/SkAAClip.cpp:836:5: warning: ‘lastY’ may be used uninitialized in this function [-Wuninitialized] ../src/core/SkAAClip.cpp: In member function ‘virtual void SkAAClipBlitter::blitV(int, int, int, SkAlpha)’: ../src/core/SkAAClip.cpp:1926:30: warning: ‘lastY’ may be used uninitialized in this function [-Wuninitialized] ../src/core/SkAAClip.cpp: In member function ‘virtual void SkAAClipBlitter::blitMask(const SkMask&, const SkIRect&)’: ../src/core/SkAAClip.cpp:2144:51: warning: ‘localStopY’ may be used uninitialized in this function [-Wuninitialized] ../src/core/SkAAClip.cpp: In member function ‘bool SkAAClip::op(const SkAAClip&, const SkAAClip&, SkRegion::Op)’: ../src/core/SkAAClip.cpp:159:9: warning: ‘iterB.SkAAClip::Iter::fStopYOff’ may be used uninitialized in this function [-Wuninitialized] ../src/core/SkAAClip.cpp:1526:20: note: ‘iterB.SkAAClip::Iter::fStopYOff’ was declared here ../src/core/SkAAClip.cpp:159:9: warning: ‘iterA.SkAAClip::Iter::fStopYOff’ may be used uninitialized in this function [-Wuninitialized] ../src/core/SkAAClip.cpp:1525:20: note: ‘iterA.SkAAClip::Iter::fStopYOff’ was declared here WHile fixing this, I observed that parameters(lastYForRow,initialCount) of findRow and findX respectively are of pointer type. The callers of these functions had to pass the parameter, even though they where not using it later.Now,the parameters are initialized to null

Patch Set 1 #

Total comments: 8

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -12 lines) Patch
M src/core/SkAAClip.h View 1 chunk +2 lines, -2 lines 0 comments Download
M src/core/SkAAClip.cpp View 1 8 chunks +11 lines, -10 lines 0 comments Download

Messages

Total messages: 6
kondapallykalyan
12 years, 4 months ago (2012-06-18 21:00:06 UTC) #1
reed1
https://codereview.appspot.com/6304098/diff/1/src/core/SkAAClip.cpp File src/core/SkAAClip.cpp (right): https://codereview.appspot.com/6304098/diff/1/src/core/SkAAClip.cpp#newcode812 src/core/SkAAClip.cpp:812: if (x < n) { Skia uses { } ...
12 years, 4 months ago (2012-06-19 15:17:14 UTC) #2
kondapallykalyan
https://codereview.appspot.com/6304098/diff/1/src/core/SkAAClip.cpp File src/core/SkAAClip.cpp (right): https://codereview.appspot.com/6304098/diff/1/src/core/SkAAClip.cpp#newcode812 src/core/SkAAClip.cpp:812: if (x < n) { On 2012/06/19 15:17:14, reed1 ...
12 years, 4 months ago (2012-06-19 16:55:57 UTC) #3
reed1
lgtm do you need me to land this?
12 years, 4 months ago (2012-06-19 17:18:06 UTC) #4
kondapallykalyan
On 2012/06/19 17:18:06, reed1 wrote: Yes..
12 years, 4 months ago (2012-06-20 08:47:43 UTC) #5
reed1
12 years, 4 months ago (2012-06-20 14:30:10 UTC) #6
landed in rev. 4283. please close this issue.
Sign in to reply to this message.

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