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

Issue 6304098: fixing compiler warnings in SkAAClip (Closed)

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