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

Issue 6202053: move more stuff that should be private out from GrGLConfig.h (Closed)

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

Patch Set 1 #

Patch Set 2 : sadjfh #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -98 lines) Patch
M include/gpu/gl/GrGLConfig.h View 1 chunk +0 lines, -85 lines 0 comments Download
M include/gpu/gl/GrGLInterface.h View 1 chunk +0 lines, -4 lines 1 comment Download
M src/gpu/GrDefaultTextContext.cpp View 1 1 chunk +1 line, -1 line 0 comments Download
M src/gpu/gl/GrGLUtil.h View 1 chunk +73 lines, -4 lines 0 comments Download
M src/gpu/gl/GrGpuGLShaders.cpp View 1 3 chunks +18 lines, -4 lines 0 comments Download

Messages

Total messages: 4
bsalomon
http://codereview.appspot.com/6202053/diff/2001/include/gpu/gl/GrGLInterface.h File include/gpu/gl/GrGLInterface.h (left): http://codereview.appspot.com/6202053/diff/2001/include/gpu/gl/GrGLInterface.h#oldcode16 include/gpu/gl/GrGLInterface.h:16: #if !defined(GR_GL_FUNCTION_TYPE) This is in the wrong place and ...
12 years, 6 months ago (2012-05-07 18:28:12 UTC) #1
TomH
Superficially LGTM.
12 years, 6 months ago (2012-05-07 18:36:25 UTC) #2
robertphillips
LGTM
12 years, 6 months ago (2012-05-07 18:44:07 UTC) #3
bsalomon
12 years, 6 months ago (2012-05-07 21:36:20 UTC) #4
Closed, r3856.
Sign in to reply to this message.

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