Commit 92d7706d by Corentin Wallez

Fix tautological compare and && in || warnings in FunctionsGL.cpp

Change-Id: Icfbd79e3cbb116da9981daeb42d3657146bf673e Reviewed-on: https://chromium-review.googlesource.com/268968Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org> Tested-by: 's avatarCorentin Wallez <cwallez@chromium.org>
parent 9946a61e
...@@ -783,7 +783,7 @@ void FunctionsGL::initialize() ...@@ -783,7 +783,7 @@ void FunctionsGL::initialize()
GetGLVersion(getString, &majorVersion, &minorVersion, &openGLES); GetGLVersion(getString, &majorVersion, &minorVersion, &openGLES);
// Grab the GL extensions // Grab the GL extensions
if (majorVersion > 3 || majorVersion == 3 && minorVersion >= 0) if (majorVersion >= 3)
{ {
AssignGLEntryPoint(loadProcAddress("glGetIntegerv"), &getIntegerv); AssignGLEntryPoint(loadProcAddress("glGetIntegerv"), &getIntegerv);
AssignGLEntryPoint(loadProcAddress("glGetStringi"), &getStringi); AssignGLEntryPoint(loadProcAddress("glGetStringi"), &getStringi);
...@@ -795,7 +795,7 @@ void FunctionsGL::initialize() ...@@ -795,7 +795,7 @@ void FunctionsGL::initialize()
} }
// 1.0 // 1.0
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 0) if (majorVersion >= 1)
{ {
AssignGLEntryPoint(loadProcAddress("glBlendFunc"), &blendFunc); AssignGLEntryPoint(loadProcAddress("glBlendFunc"), &blendFunc);
AssignGLEntryPoint(loadProcAddress("glClear"), &clear); AssignGLEntryPoint(loadProcAddress("glClear"), &clear);
...@@ -848,7 +848,7 @@ void FunctionsGL::initialize() ...@@ -848,7 +848,7 @@ void FunctionsGL::initialize()
} }
// 1.1 // 1.1
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 1) if (majorVersion > 1 || (majorVersion == 1 && minorVersion >= 1))
{ {
AssignGLEntryPoint(loadProcAddress("glBindTexture"), &bindTexture); AssignGLEntryPoint(loadProcAddress("glBindTexture"), &bindTexture);
AssignGLEntryPoint(loadProcAddress("glCopyTexImage1D"), &copyTexImage1D); AssignGLEntryPoint(loadProcAddress("glCopyTexImage1D"), &copyTexImage1D);
...@@ -866,7 +866,7 @@ void FunctionsGL::initialize() ...@@ -866,7 +866,7 @@ void FunctionsGL::initialize()
} }
// 1.2 // 1.2
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 2) if (majorVersion > 1 || (majorVersion == 1 && minorVersion >= 2))
{ {
AssignGLEntryPoint(loadProcAddress("glBlendColor"), &blendColor); AssignGLEntryPoint(loadProcAddress("glBlendColor"), &blendColor);
AssignGLEntryPoint(loadProcAddress("glBlendEquation"), &blendEquation); AssignGLEntryPoint(loadProcAddress("glBlendEquation"), &blendEquation);
...@@ -886,7 +886,7 @@ void FunctionsGL::initialize() ...@@ -886,7 +886,7 @@ void FunctionsGL::initialize()
} }
// 1.3 // 1.3
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 3) if (majorVersion > 1 || (majorVersion == 1 && minorVersion >= 3))
{ {
AssignGLEntryPoint(loadProcAddress("glActiveTexture"), &activeTexture); AssignGLEntryPoint(loadProcAddress("glActiveTexture"), &activeTexture);
AssignGLEntryPoint(loadProcAddress("glCompressedTexImage1D"), &compressedTexImage1D); AssignGLEntryPoint(loadProcAddress("glCompressedTexImage1D"), &compressedTexImage1D);
...@@ -900,7 +900,7 @@ void FunctionsGL::initialize() ...@@ -900,7 +900,7 @@ void FunctionsGL::initialize()
} }
// 1.4 // 1.4
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 4) if (majorVersion > 1 || (majorVersion == 1 && minorVersion >= 4))
{ {
AssignGLEntryPoint(loadProcAddress("glBlendFuncSeparate"), &blendFuncSeparate); AssignGLEntryPoint(loadProcAddress("glBlendFuncSeparate"), &blendFuncSeparate);
AssignGLEntryPoint(loadProcAddress("glMultiDrawArrays"), &multiDrawArrays); AssignGLEntryPoint(loadProcAddress("glMultiDrawArrays"), &multiDrawArrays);
...@@ -912,7 +912,7 @@ void FunctionsGL::initialize() ...@@ -912,7 +912,7 @@ void FunctionsGL::initialize()
} }
// 1.5 // 1.5
if (majorVersion > 1 || majorVersion == 1 && minorVersion >= 5) if (majorVersion > 1 || (majorVersion == 1 && minorVersion >= 5))
{ {
AssignGLEntryPoint(loadProcAddress("glBeginQuery"), &beginQuery); AssignGLEntryPoint(loadProcAddress("glBeginQuery"), &beginQuery);
AssignGLEntryPoint(loadProcAddress("glBindBuffer"), &bindBuffer); AssignGLEntryPoint(loadProcAddress("glBindBuffer"), &bindBuffer);
...@@ -936,7 +936,7 @@ void FunctionsGL::initialize() ...@@ -936,7 +936,7 @@ void FunctionsGL::initialize()
} }
// 2.0 // 2.0
if (majorVersion > 2 || majorVersion == 2 && minorVersion >= 0) if (majorVersion >= 2)
{ {
AssignGLEntryPoint(loadProcAddress("glAttachShader"), &attachShader); AssignGLEntryPoint(loadProcAddress("glAttachShader"), &attachShader);
AssignGLEntryPoint(loadProcAddress("glBindAttribLocation"), &bindAttribLocation); AssignGLEntryPoint(loadProcAddress("glBindAttribLocation"), &bindAttribLocation);
...@@ -1034,7 +1034,7 @@ void FunctionsGL::initialize() ...@@ -1034,7 +1034,7 @@ void FunctionsGL::initialize()
} }
// 2.1 // 2.1
if (majorVersion > 2 || majorVersion == 2 && minorVersion >= 1) if (majorVersion > 2 || (majorVersion == 2 && minorVersion >= 1))
{ {
AssignGLEntryPoint(loadProcAddress("glUniformMatrix2x3fv"), &uniformMatrix2x3fv); AssignGLEntryPoint(loadProcAddress("glUniformMatrix2x3fv"), &uniformMatrix2x3fv);
AssignGLEntryPoint(loadProcAddress("glUniformMatrix2x4fv"), &uniformMatrix2x4fv); AssignGLEntryPoint(loadProcAddress("glUniformMatrix2x4fv"), &uniformMatrix2x4fv);
...@@ -1045,7 +1045,7 @@ void FunctionsGL::initialize() ...@@ -1045,7 +1045,7 @@ void FunctionsGL::initialize()
} }
// 3.0 // 3.0
if (majorVersion > 3 || majorVersion == 3 && minorVersion >= 0) if (majorVersion >= 3)
{ {
AssignGLEntryPoint(loadProcAddress("glBeginConditionalRender"), &beginConditionalRender); AssignGLEntryPoint(loadProcAddress("glBeginConditionalRender"), &beginConditionalRender);
AssignGLEntryPoint(loadProcAddress("glBeginTransformFeedback"), &beginTransformFeedback); AssignGLEntryPoint(loadProcAddress("glBeginTransformFeedback"), &beginTransformFeedback);
...@@ -1137,7 +1137,7 @@ void FunctionsGL::initialize() ...@@ -1137,7 +1137,7 @@ void FunctionsGL::initialize()
} }
// 3.1 // 3.1
if (majorVersion > 3 || majorVersion == 3 && minorVersion >= 1) if (majorVersion > 3 || (majorVersion == 3 && minorVersion >= 1))
{ {
AssignGLEntryPoint(loadProcAddress("glCopyBufferSubData"), &copyBufferSubData); AssignGLEntryPoint(loadProcAddress("glCopyBufferSubData"), &copyBufferSubData);
AssignGLEntryPoint(loadProcAddress("glDrawArraysInstanced"), &drawArraysInstanced); AssignGLEntryPoint(loadProcAddress("glDrawArraysInstanced"), &drawArraysInstanced);
...@@ -1164,7 +1164,7 @@ void FunctionsGL::initialize() ...@@ -1164,7 +1164,7 @@ void FunctionsGL::initialize()
} }
// 3.2 // 3.2
if (majorVersion > 3 || majorVersion == 3 && minorVersion >= 2) if (majorVersion > 3 || (majorVersion == 3 && minorVersion >= 2))
{ {
AssignGLEntryPoint(loadProcAddress("glClientWaitSync"), &clientWaitSync); AssignGLEntryPoint(loadProcAddress("glClientWaitSync"), &clientWaitSync);
AssignGLEntryPoint(loadProcAddress("glDeleteSync"), &deleteSync); AssignGLEntryPoint(loadProcAddress("glDeleteSync"), &deleteSync);
...@@ -1188,7 +1188,7 @@ void FunctionsGL::initialize() ...@@ -1188,7 +1188,7 @@ void FunctionsGL::initialize()
} }
// 3.3 // 3.3
if (majorVersion > 3 || majorVersion == 3 && minorVersion >= 3) if (majorVersion > 3 || (majorVersion == 3 && minorVersion >= 3))
{ {
AssignGLEntryPoint(loadProcAddress("glBindFragDataLocationIndexed"), &bindFragDataLocationIndexed); AssignGLEntryPoint(loadProcAddress("glBindFragDataLocationIndexed"), &bindFragDataLocationIndexed);
AssignGLEntryPoint(loadProcAddress("glBindSampler"), &bindSampler); AssignGLEntryPoint(loadProcAddress("glBindSampler"), &bindSampler);
...@@ -1221,7 +1221,7 @@ void FunctionsGL::initialize() ...@@ -1221,7 +1221,7 @@ void FunctionsGL::initialize()
} }
// 4.0 // 4.0
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 0) if (majorVersion >= 4)
{ {
AssignGLEntryPoint(loadProcAddress("glBeginQueryIndexed"), &beginQueryIndexed); AssignGLEntryPoint(loadProcAddress("glBeginQueryIndexed"), &beginQueryIndexed);
AssignGLEntryPoint(loadProcAddress("glBindTransformFeedback"), &bindTransformFeedback); AssignGLEntryPoint(loadProcAddress("glBindTransformFeedback"), &bindTransformFeedback);
...@@ -1271,8 +1271,8 @@ void FunctionsGL::initialize() ...@@ -1271,8 +1271,8 @@ void FunctionsGL::initialize()
AssignGLEntryPoint(loadProcAddress("glUniformSubroutinesuiv"), &uniformSubroutinesuiv); AssignGLEntryPoint(loadProcAddress("glUniformSubroutinesuiv"), &uniformSubroutinesuiv);
} }
// 4.0 // 4.1
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 1) if (majorVersion > 4 || (majorVersion == 4 && minorVersion >= 1))
{ {
AssignGLEntryPoint(loadProcAddress("glActiveShaderProgram"), &activeShaderProgram); AssignGLEntryPoint(loadProcAddress("glActiveShaderProgram"), &activeShaderProgram);
AssignGLEntryPoint(loadProcAddress("glBindProgramPipeline"), &bindProgramPipeline); AssignGLEntryPoint(loadProcAddress("glBindProgramPipeline"), &bindProgramPipeline);
...@@ -1365,7 +1365,7 @@ void FunctionsGL::initialize() ...@@ -1365,7 +1365,7 @@ void FunctionsGL::initialize()
} }
// 4.2 // 4.2
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 2) if (majorVersion > 4 || (majorVersion == 4 && minorVersion >= 2))
{ {
AssignGLEntryPoint(loadProcAddress("glBindImageTexture"), &bindImageTexture); AssignGLEntryPoint(loadProcAddress("glBindImageTexture"), &bindImageTexture);
AssignGLEntryPoint(loadProcAddress("glDrawArraysInstancedBaseInstance"), &drawArraysInstancedBaseInstance); AssignGLEntryPoint(loadProcAddress("glDrawArraysInstancedBaseInstance"), &drawArraysInstancedBaseInstance);
...@@ -1382,7 +1382,7 @@ void FunctionsGL::initialize() ...@@ -1382,7 +1382,7 @@ void FunctionsGL::initialize()
} }
// 4.3 // 4.3
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 3) if (majorVersion > 4 || (majorVersion == 4 && minorVersion >= 3))
{ {
AssignGLEntryPoint(loadProcAddress("glBindVertexBuffer"), &bindVertexBuffer); AssignGLEntryPoint(loadProcAddress("glBindVertexBuffer"), &bindVertexBuffer);
AssignGLEntryPoint(loadProcAddress("glClearBufferData"), &clearBufferData); AssignGLEntryPoint(loadProcAddress("glClearBufferData"), &clearBufferData);
...@@ -1430,7 +1430,7 @@ void FunctionsGL::initialize() ...@@ -1430,7 +1430,7 @@ void FunctionsGL::initialize()
} }
// 4.4 // 4.4
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 4) if (majorVersion > 4 || (majorVersion == 4 && minorVersion >= 4))
{ {
AssignGLEntryPoint(loadProcAddress("glBindBuffersBase"), &bindBuffersBase); AssignGLEntryPoint(loadProcAddress("glBindBuffersBase"), &bindBuffersBase);
AssignGLEntryPoint(loadProcAddress("glBindBuffersRange"), &bindBuffersRange); AssignGLEntryPoint(loadProcAddress("glBindBuffersRange"), &bindBuffersRange);
...@@ -1444,7 +1444,7 @@ void FunctionsGL::initialize() ...@@ -1444,7 +1444,7 @@ void FunctionsGL::initialize()
} }
// 4.5 // 4.5
if (majorVersion > 4 || majorVersion == 4 && minorVersion >= 5) if (majorVersion > 4 || (majorVersion == 4 && minorVersion >= 5))
{ {
AssignGLEntryPoint(loadProcAddress("glBindTextureUnit"), &bindTextureUnit); AssignGLEntryPoint(loadProcAddress("glBindTextureUnit"), &bindTextureUnit);
AssignGLEntryPoint(loadProcAddress("glBlitNamedFramebuffer"), &blitNamedFramebuffer); AssignGLEntryPoint(loadProcAddress("glBlitNamedFramebuffer"), &blitNamedFramebuffer);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment