diff options
| author | Sergey Sharybin <sergey.vfx@gmail.com> | 2017-05-08 17:40:48 +0300 |
|---|---|---|
| committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2017-05-08 17:40:59 +0300 |
| commit | 883c62d5d3c896b44d8e9503629c1079654fb284 (patch) | |
| tree | 2e10a9e24ad32edd7d0beeb4d345ced1665e346f /intern/opencolorio | |
| parent | 6aa7843de7d579724cb0cea0c66a5ddc4c03c959 (diff) | |
Add comment about vertex format created in OCIO
Diffstat (limited to 'intern/opencolorio')
| -rw-r--r-- | intern/opencolorio/ocio_impl_glsl.cc | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/intern/opencolorio/ocio_impl_glsl.cc b/intern/opencolorio/ocio_impl_glsl.cc index 8323cffd09b..89df3080749 100644 --- a/intern/opencolorio/ocio_impl_glsl.cc +++ b/intern/opencolorio/ocio_impl_glsl.cc @@ -416,7 +416,14 @@ bool OCIOImpl::setupGLSLDraw(OCIO_GLSLDrawState **state_r, OCIO_ConstProcessorRc glActiveTexture(GL_TEXTURE0); - /* IMM needs vertex format even if we don't draw with it */ + /* IMM needs vertex format even if we don't draw with it. + * + * NOTE: The only reason why it's here is because of Cycles viewport. + * All other areas are managing their own vertex formats. + * Doing it here is probably harmless, but kind of stupid. + * + * TODO(sergey): Look into some nicer solution. + */ VertexFormat *format = immVertexFormat(); VertexFormat_add_attrib(format, "pos", COMP_F32, 2, KEEP_FLOAT); VertexFormat_add_attrib(format, "texCoord", COMP_F32, 2, KEEP_FLOAT); |