Unify framebuffer api naming.

This commit is contained in:
Paul Chote
2015-12-28 11:02:22 +00:00
parent 4372ed650e
commit e63fc00b76
9 changed files with 108 additions and 126 deletions

View File

@@ -1,29 +0,0 @@
#region Copyright & License Information
/*
* Copyright 2007-2015 The OpenRA Developers (see AUTHORS)
* This file is part of OpenRA, which is free software. It is made
* available to you under the terms of the GNU General Public License
* as published by the Free Software Foundation. For more information,
* see COPYING.
*/
#endregion
using System;
using System.Diagnostics;
namespace OpenRA.Platforms.Default
{
static class ErrorHandler
{
public static void CheckGlError()
{
var n = OpenGL.glGetError();
if (n != OpenGL.GL_NO_ERROR)
{
var error = "GL Error: {0}\n{1}".F(n, new StackTrace());
OpenGL.WriteGraphicsLog(error);
throw new InvalidOperationException("OpenGL Error: See graphics.log for details.");
}
}
}
}

View File

@@ -29,28 +29,28 @@ namespace OpenRA.Platforms.Default
throw new InvalidDataException("Frame buffer size ({0}x{1}) must be a power of two".F(size.Width, size.Height));
OpenGL.glGenFramebuffersEXT(1, out framebuffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindFramebufferEXT(OpenGL.FRAMEBUFFER_EXT, framebuffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
// Color
texture = new Texture();
texture.SetEmpty(size.Width, size.Height);
OpenGL.glFramebufferTexture2DEXT(OpenGL.FRAMEBUFFER_EXT, OpenGL.COLOR_ATTACHMENT0_EXT, OpenGL.GL_TEXTURE_2D, texture.ID, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
// Depth
OpenGL.glGenRenderbuffersEXT(1, out depth);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindRenderbufferEXT(OpenGL.RENDERBUFFER_EXT, depth);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glRenderbufferStorageEXT(OpenGL.RENDERBUFFER_EXT, OpenGL.GL_DEPTH_COMPONENT, size.Width, size.Height);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glFramebufferRenderbufferEXT(OpenGL.FRAMEBUFFER_EXT, OpenGL.DEPTH_ATTACHMENT_EXT, OpenGL.RENDERBUFFER_EXT, depth);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
// Test for completeness
var status = OpenGL.glCheckFramebufferStatus(OpenGL.FRAMEBUFFER_EXT);
@@ -63,7 +63,7 @@ namespace OpenRA.Platforms.Default
// Restore default buffer
OpenGL.glBindFramebufferEXT(OpenGL.FRAMEBUFFER_EXT, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
static int[] ViewportRectangle()
@@ -75,7 +75,7 @@ namespace OpenRA.Platforms.Default
OpenGL.glGetIntegerv(OpenGL.GL_VIEWPORT, ptr);
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
return v;
}
@@ -88,26 +88,26 @@ namespace OpenRA.Platforms.Default
cv = ViewportRectangle();
OpenGL.glFlush();
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindFramebufferEXT(OpenGL.FRAMEBUFFER_EXT, framebuffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glViewport(0, 0, size.Width, size.Height);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glClearColor(0, 0, 0, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glClear(OpenGL.GL_COLOR_BUFFER_BIT | OpenGL.GL_DEPTH_BUFFER_BIT);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void Unbind()
{
VerifyThreadAffinity();
OpenGL.glFlush();
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindFramebufferEXT(OpenGL.FRAMEBUFFER_EXT, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glViewport(cv[0], cv[1], cv[2], cv[3]);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public ITexture Texture
@@ -139,9 +139,9 @@ namespace OpenRA.Platforms.Default
texture.Dispose();
OpenGL.glDeleteFramebuffersEXT(1, ref framebuffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glDeleteRenderbuffersEXT(1, ref depth);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
}
}

View File

@@ -9,6 +9,7 @@
#endregion
using System;
using System.Diagnostics;
using System.Diagnostics.CodeAnalysis;
using System.Runtime.InteropServices;
using System.Text;
@@ -439,7 +440,18 @@ namespace OpenRA.Platforms.Default
throw new InvalidProgramException("OpenGL Version Error: See graphics.log for details.");
}
ErrorHandler.CheckGlError();
CheckGlError();
}
public static void CheckGLError()
{
var n = OpenGL.glGetError();
if (n != OpenGL.GL_NO_ERROR)
{
var error = "GL Error: {0}\n{1}".F(n, new StackTrace());
WriteGraphicsLog(error);
throw new InvalidOperationException("OpenGL Error: See graphics.log for details.");
}
}
public static void WriteGraphicsLog(string message)

View File

@@ -49,7 +49,6 @@
<Compile Include="Sdl2GraphicsDevice.cs" />
<Compile Include="Sdl2Input.cs" />
<Compile Include="Shader.cs" />
<Compile Include="ErrorHandler.cs" />
<Compile Include="FrameBuffer.cs" />
<Compile Include="MultiTapDetection.cs" />
<Compile Include="Texture.cs" />

View File

@@ -79,9 +79,9 @@ namespace OpenRA.Platforms.Default
OpenGL.Initialize();
OpenGL.glEnableVertexAttribArray(Shader.VertexPosAttributeIndex);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glEnableVertexAttribArray(Shader.TexCoordAttributeIndex);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
SDL.SDL_SetModState(SDL.SDL_Keymod.KMOD_NONE);
input = new Sdl2Input();
@@ -206,39 +206,39 @@ namespace OpenRA.Platforms.Default
{
VerifyThreadAffinity();
OpenGL.glDrawArrays(ModeFromPrimitiveType(pt), firstVertex, numVertices);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void Clear()
{
VerifyThreadAffinity();
OpenGL.glClearColor(0, 0, 0, 1);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glClear(OpenGL.GL_COLOR_BUFFER_BIT);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void EnableDepthBuffer()
{
VerifyThreadAffinity();
OpenGL.glClear(OpenGL.GL_DEPTH_BUFFER_BIT);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glEnable(OpenGL.GL_DEPTH_TEST);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void DisableDepthBuffer()
{
VerifyThreadAffinity();
OpenGL.glDisable(OpenGL.GL_DEPTH_TEST);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetBlendMode(BlendMode mode)
{
VerifyThreadAffinity();
OpenGL.glBlendEquation(OpenGL.GL_FUNC_ADD);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
switch (mode)
{
@@ -247,40 +247,40 @@ namespace OpenRA.Platforms.Default
break;
case BlendMode.Alpha:
OpenGL.glEnable(OpenGL.GL_BLEND);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendFunc(OpenGL.GL_ONE, OpenGL.GL_ONE_MINUS_SRC_ALPHA);
break;
case BlendMode.Additive:
case BlendMode.Subtractive:
OpenGL.glEnable(OpenGL.GL_BLEND);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendFunc(OpenGL.GL_ONE, OpenGL.GL_ONE);
if (mode == BlendMode.Subtractive)
{
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendEquation(OpenGL.GL_FUNC_REVERSE_SUBTRACT);
}
break;
case BlendMode.Multiply:
OpenGL.glEnable(OpenGL.GL_BLEND);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendFunc(OpenGL.GL_DST_COLOR, OpenGL.GL_ONE_MINUS_SRC_ALPHA);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
break;
case BlendMode.Multiplicative:
OpenGL.glEnable(OpenGL.GL_BLEND);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendFunc(OpenGL.GL_ZERO, OpenGL.GL_SRC_COLOR);
break;
case BlendMode.DoubleMultiplicative:
OpenGL.glEnable(OpenGL.GL_BLEND);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBlendFunc(OpenGL.GL_DST_COLOR, OpenGL.GL_SRC_COLOR);
break;
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void GrabWindowMouseFocus()
@@ -306,16 +306,16 @@ namespace OpenRA.Platforms.Default
height = 0;
OpenGL.glScissor(left, WindowSize.Height - (top + height), width, height);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glEnable(OpenGL.GL_SCISSOR_TEST);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void DisableScissor()
{
VerifyThreadAffinity();
OpenGL.glDisable(OpenGL.GL_SCISSOR_TEST);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public Bitmap TakeScreenshot()

View File

@@ -179,7 +179,7 @@ namespace OpenRA.Platforms.Default
pendingMotion = null;
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
}
}

View File

@@ -31,19 +31,19 @@ namespace OpenRA.Platforms.Default
var code = File.ReadAllText(filename);
var shader = OpenGL.glCreateShader(type);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
unsafe
{
var length = code.Length;
OpenGL.glShaderSource(shader, 1, new string[] { code }, new IntPtr(&length));
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glCompileShader(shader);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
int success;
OpenGL.glGetShaderiv(shader, OpenGL.GL_COMPILE_STATUS, out success);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
if (success == (int)OpenGL.GL_FALSE)
{
int len;
@@ -66,23 +66,23 @@ namespace OpenRA.Platforms.Default
// Assemble program
program = OpenGL.glCreateProgram();
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindAttribLocation(program, VertexPosAttributeIndex, "aVertexPosition");
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindAttribLocation(program, TexCoordAttributeIndex, "aVertexTexCoord");
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glAttachShader(program, vertexShader);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glAttachShader(program, fragmentShader);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glLinkProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
int success;
OpenGL.glGetProgramiv(program, OpenGL.GL_LINK_STATUS, out success);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
if (success == (int)OpenGL.GL_FALSE)
{
int len;
@@ -96,12 +96,12 @@ namespace OpenRA.Platforms.Default
}
OpenGL.glUseProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
int numUniforms;
OpenGL.glGetProgramiv(program, OpenGL.GL_ACTIVE_UNIFORMS, out numUniforms);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var nextTexUnit = 0;
for (var i = 0; i < numUniforms; i++)
@@ -111,16 +111,16 @@ namespace OpenRA.Platforms.Default
var sb = new StringBuilder(128);
OpenGL.glGetActiveUniform(program, i, 128, out length, out size, out type, sb);
var sampler = sb.ToString();
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
if (type == OpenGL.GL_SAMPLER_2D)
{
samplers.Add(sampler, nextTexUnit);
var loc = OpenGL.glGetUniformLocation(program, sampler);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glUniform1i(loc, nextTexUnit);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
nextTexUnit++;
}
@@ -139,9 +139,9 @@ namespace OpenRA.Platforms.Default
OpenGL.glBindTexture(OpenGL.GL_TEXTURE_2D, ((Texture)kv.Value).ID);
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
a();
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetTexture(string name, ITexture t)
@@ -159,40 +159,40 @@ namespace OpenRA.Platforms.Default
{
VerifyThreadAffinity();
OpenGL.glUseProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var param = OpenGL.glGetUniformLocation(program, name);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glUniform1i(param, value ? 1 : 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetVec(string name, float x)
{
VerifyThreadAffinity();
OpenGL.glUseProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var param = OpenGL.glGetUniformLocation(program, name);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glUniform1f(param, x);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetVec(string name, float x, float y)
{
VerifyThreadAffinity();
OpenGL.glUseProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var param = OpenGL.glGetUniformLocation(program, name);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glUniform2f(param, x, y);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetVec(string name, float[] vec, int length)
{
VerifyThreadAffinity();
var param = OpenGL.glGetUniformLocation(program, name);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
unsafe
{
fixed (float* pVec = vec)
@@ -209,7 +209,7 @@ namespace OpenRA.Platforms.Default
}
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetMatrix(string name, float[] mtx)
@@ -219,9 +219,9 @@ namespace OpenRA.Platforms.Default
throw new InvalidDataException("Invalid 4x4 matrix");
OpenGL.glUseProgram(program);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var param = OpenGL.glGetUniformLocation(program, name);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
unsafe
{
@@ -229,7 +229,7 @@ namespace OpenRA.Platforms.Default
OpenGL.glUniformMatrix4fv(param, 1, false, new IntPtr(pMtx));
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
}
}

View File

@@ -46,37 +46,37 @@ namespace OpenRA.Platforms.Default
public Texture()
{
OpenGL.glGenTextures(1, out texture);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public Texture(Bitmap bitmap)
{
OpenGL.glGenTextures(1, out texture);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
SetData(bitmap);
}
void PrepareTexture()
{
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindTexture(OpenGL.GL_TEXTURE_2D, texture);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
var filter = scaleFilter == TextureScaleFilter.Linear ? OpenGL.GL_LINEAR : OpenGL.GL_NEAREST;
OpenGL.glTexParameteri(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_MAG_FILTER, (int)filter);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glTexParameteri(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_MIN_FILTER, (int)filter);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glTexParameterf(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_WRAP_S, (float)OpenGL.GL_CLAMP_TO_EDGE);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glTexParameterf(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_WRAP_T, (float)OpenGL.GL_CLAMP_TO_EDGE);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glTexParameteri(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_BASE_LEVEL, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glTexParameteri(OpenGL.GL_TEXTURE_2D, OpenGL.GL_TEXTURE_MAX_LEVEL, 0);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetData(byte[] colors, int width, int height)
@@ -94,7 +94,7 @@ namespace OpenRA.Platforms.Default
PrepareTexture();
OpenGL.glTexImage2D(OpenGL.GL_TEXTURE_2D, 0, OpenGL.GL_RGBA8, width, height,
0, OpenGL.GL_BGRA, OpenGL.GL_UNSIGNED_BYTE, intPtr);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
}
}
@@ -118,7 +118,7 @@ namespace OpenRA.Platforms.Default
PrepareTexture();
OpenGL.glTexImage2D(OpenGL.GL_TEXTURE_2D, 0, OpenGL.GL_RGBA8, width, height,
0, OpenGL.GL_BGRA, OpenGL.GL_UNSIGNED_BYTE, intPtr);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
}
}
@@ -142,7 +142,7 @@ namespace OpenRA.Platforms.Default
PrepareTexture();
OpenGL.glTexImage2D(OpenGL.GL_TEXTURE_2D, 0, OpenGL.GL_RGBA8, bits.Width, bits.Height,
0, OpenGL.GL_BGRA, OpenGL.GL_UNSIGNED_BYTE, bits.Scan0); // TODO: weird strides
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
bitmap.UnlockBits(bits);
}
finally
@@ -157,7 +157,7 @@ namespace OpenRA.Platforms.Default
VerifyThreadAffinity();
var data = new byte[4 * Size.Width * Size.Height];
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glBindTexture(OpenGL.GL_TEXTURE_2D, texture);
unsafe
{
@@ -169,7 +169,7 @@ namespace OpenRA.Platforms.Default
}
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
return data;
}
@@ -183,7 +183,7 @@ namespace OpenRA.Platforms.Default
PrepareTexture();
OpenGL.glTexImage2D(OpenGL.GL_TEXTURE_2D, 0, OpenGL.GL_RGBA8, width, height,
0, OpenGL.GL_BGRA, OpenGL.GL_UNSIGNED_BYTE, IntPtr.Zero);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
~Texture()

View File

@@ -23,7 +23,7 @@ namespace OpenRA.Platforms.Default
public VertexBuffer(int size)
{
OpenGL.glGenBuffers(1, out buffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
Bind();
var ptr = GCHandle.Alloc(new T[size], GCHandleType.Pinned);
@@ -39,7 +39,7 @@ namespace OpenRA.Platforms.Default
ptr.Free();
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetData(T[] data, int length)
@@ -64,7 +64,7 @@ namespace OpenRA.Platforms.Default
ptr.Free();
}
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void SetData(IntPtr data, int start, int length)
@@ -74,18 +74,18 @@ namespace OpenRA.Platforms.Default
new IntPtr(VertexSize * start),
new IntPtr(VertexSize * length),
data);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
public void Bind()
{
VerifyThreadAffinity();
OpenGL.glBindBuffer(OpenGL.GL_ARRAY_BUFFER, buffer);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glVertexAttribPointer(Shader.VertexPosAttributeIndex, 3, OpenGL.GL_FLOAT, false, VertexSize, IntPtr.Zero);
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
OpenGL.glVertexAttribPointer(Shader.TexCoordAttributeIndex, 4, OpenGL.GL_FLOAT, false, VertexSize, new IntPtr(12));
ErrorHandler.CheckGlError();
OpenGL.CheckGLError();
}
~VertexBuffer()