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

Issue 4837068: Fix some ref leaks (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by bsalomon
Modified:
13 years, 3 months ago
Reviewers:
bungeman
Base URL:
http://skia.googlecode.com/svn/trunk/gpu/src/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : correct files #

Patch Set 3 : use SkAutoTUnref rather than SkRefPtr #

Patch Set 4 : better #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -11 lines) Patch
M gpu/src/GrGpuGL.cpp View 1 2 3 7 chunks +10 lines, -10 lines 0 comments Download
M include/core/SkRefCnt.h View 1 2 1 chunk +6 lines, -1 line 0 comments Download
M src/gpu/SkGpuDevice.cpp View 1 2 3 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3
bsalomon
13 years, 3 months ago (2011-08-09 14:32:31 UTC) #1
bungeman
On 2011/08/09 14:32:31, bsalomon wrote: LGTM
13 years, 3 months ago (2011-08-09 15:02:39 UTC) #2
bsalomon
13 years, 3 months ago (2011-08-10 12:11:34 UTC) #3
On 2011/08/09 15:02:39, bungeman wrote:
> On 2011/08/09 14:32:31, bsalomon wrote:
> 
> LGTM

Closed with r2082.
Sign in to reply to this message.

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