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

Issue 6304098: fixing compiler warnings in SkAAClip (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years ago by kondapallykalyan
Modified:
13 years 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
13 years 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 { } ...
13 years 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 ...
13 years ago (2012-06-19 16:55:57 UTC) #3
reed1
lgtm do you need me to land this?
13 years ago (2012-06-19 17:18:06 UTC) #4
kondapallykalyan
On 2012/06/19 17:18:06, reed1 wrote: Yes..
13 years ago (2012-06-20 08:47:43 UTC) #5
reed1
13 years 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