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

Issue 4185056: Remove glGet calls for stencil bits, also cleanup gl vs. gr irects (Closed)

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

Patch Set 1 #

Patch Set 2 : fix comment #

Patch Set 3 : static assert #

Unified diffs Side-by-side diffs Delta from patch set Stats (+222 lines, -139 lines) Patch
M gpu/include/GrContext.h View 1 chunk +8 lines, -1 line 0 comments Download
M gpu/include/GrGLConfig.h View 1 2 1 chunk +96 lines, -0 lines 0 comments Download
M gpu/include/GrGLTexture.h View 5 chunks +25 lines, -22 lines 0 comments Download
M gpu/include/GrGpu.h View 1 2 chunks +2 lines, -6 lines 0 comments Download
M gpu/include/GrTexture.h View 1 chunk +2 lines, -2 lines 0 comments Download
M gpu/include/GrUserConfig.h View 1 chunk +0 lines, -2 lines 0 comments Download
M gpu/src/GrContext.cpp View 1 chunk +5 lines, -3 lines 0 comments Download
M gpu/src/GrGLTexture.cpp View 2 chunks +12 lines, -11 lines 0 comments Download
M gpu/src/GrGpuGL.h View 3 chunks +7 lines, -29 lines 0 comments Download
M gpu/src/GrGpuGL.cpp View 13 chunks +65 lines, -63 lines 0 comments Download

Messages

Total messages: 3
bsalomon
13 years, 8 months ago (2011-02-18 14:48:54 UTC) #1
reed1
Can GrGLIRect be in its own header? LGTM
13 years, 8 months ago (2011-02-18 15:40:28 UTC) #2
bsalomon
13 years, 8 months ago (2011-02-18 16:10:11 UTC) #3
On 2011/02/18 15:40:28, reed1 wrote:
> Can GrGLIRect be in its own header?

Done, and checked in.

> 
> LGTM
Sign in to reply to this message.

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