aboutsummaryrefslogtreecommitdiffstats
path: root/emulator
diff options
context:
space:
mode:
Diffstat (limited to 'emulator')
-rw-r--r--emulator/opengl/host/libs/libOpenglRender/FrameBuffer.cpp4
-rw-r--r--emulator/opengl/host/libs/libOpenglRender/RenderControl.cpp4
-rw-r--r--emulator/opengl/host/libs/libOpenglRender/WindowSurface.cpp23
-rw-r--r--emulator/opengl/host/libs/libOpenglRender/WindowSurface.h4
4 files changed, 21 insertions, 14 deletions
diff --git a/emulator/opengl/host/libs/libOpenglRender/FrameBuffer.cpp b/emulator/opengl/host/libs/libOpenglRender/FrameBuffer.cpp
index 2c9e8c5..8609e09 100644
--- a/emulator/opengl/host/libs/libOpenglRender/FrameBuffer.cpp
+++ b/emulator/opengl/host/libs/libOpenglRender/FrameBuffer.cpp
@@ -578,9 +578,7 @@ bool FrameBuffer::flushWindowSurfaceColorBuffer(HandleType p_surface)
return false;
}
- (*w).second->flushColorBuffer();
-
- return true;
+ return (*w).second->flushColorBuffer();
}
bool FrameBuffer::setWindowSurfaceColorBuffer(HandleType p_surface,
diff --git a/emulator/opengl/host/libs/libOpenglRender/RenderControl.cpp b/emulator/opengl/host/libs/libOpenglRender/RenderControl.cpp
index 960dec7..6a15138 100644
--- a/emulator/opengl/host/libs/libOpenglRender/RenderControl.cpp
+++ b/emulator/opengl/host/libs/libOpenglRender/RenderControl.cpp
@@ -239,7 +239,9 @@ static int rcFlushWindowColorBuffer(uint32_t windowSurface)
if (!fb) {
return -1;
}
- fb->flushWindowSurfaceColorBuffer(windowSurface);
+ if (!fb->flushWindowSurfaceColorBuffer(windowSurface)) {
+ return -1;
+ }
return 0;
}
diff --git a/emulator/opengl/host/libs/libOpenglRender/WindowSurface.cpp b/emulator/opengl/host/libs/libOpenglRender/WindowSurface.cpp
index 3674120..9c32ff8 100644
--- a/emulator/opengl/host/libs/libOpenglRender/WindowSurface.cpp
+++ b/emulator/opengl/host/libs/libOpenglRender/WindowSurface.cpp
@@ -81,11 +81,12 @@ WindowSurface *WindowSurface::create(int p_config, int p_width, int p_height)
// previous attached color buffer is updated, if copy or blit should be done
// in order to update it - it is being done here.
//
-void WindowSurface::flushColorBuffer()
+bool WindowSurface::flushColorBuffer()
{
if (m_attachedColorBuffer.Ptr() != NULL) {
- blitToColorBuffer();
+ return blitToColorBuffer();
}
+ return true;
}
//
@@ -140,14 +141,15 @@ void WindowSurface::bind(RenderContextPtr p_ctx, SurfaceBindType p_bindType)
}
-void WindowSurface::blitToColorBuffer()
+bool WindowSurface::blitToColorBuffer()
{
- if (!m_width && !m_height) return;
+ if (!m_width && !m_height) return false;
if (m_attachedColorBuffer->getWidth() != m_width ||
m_attachedColorBuffer->getHeight() != m_height) {
// XXX: should never happen - how this needs to be handled?
- return;
+ fprintf(stderr, "Dimensions do not match\n");
+ return false;
}
//
@@ -157,9 +159,14 @@ void WindowSurface::blitToColorBuffer()
EGLSurface prevReadSurf = s_egl.eglGetCurrentSurface(EGL_READ);
EGLSurface prevDrawSurf = s_egl.eglGetCurrentSurface(EGL_DRAW);
FrameBuffer *fb = FrameBuffer::getFB();
+ if (!m_drawContext.Ptr()) {
+ fprintf(stderr, "Draw context is NULL\n");
+ return false;
+ }
if (!s_egl.eglMakeCurrent(fb->getDisplay(), m_eglSurface,
m_eglSurface, m_drawContext->getEGLContext())) {
- return;
+ fprintf(stderr, "Error making draw context current\n");
+ return false;
}
m_attachedColorBuffer->blitFromCurrentReadBuffer();
@@ -167,12 +174,12 @@ void WindowSurface::blitToColorBuffer()
// restore current context/surface
s_egl.eglMakeCurrent(fb->getDisplay(), prevDrawSurf,
prevReadSurf, prevContext);
-
+ return true;
}
bool WindowSurface::resizePbuffer(unsigned int p_width, unsigned int p_height)
{
- if (m_eglSurface &&
+ if (m_eglSurface &&
m_pbufWidth == p_width &&
m_pbufHeight == p_height) {
// no need to resize
diff --git a/emulator/opengl/host/libs/libOpenglRender/WindowSurface.h b/emulator/opengl/host/libs/libOpenglRender/WindowSurface.h
index 1b655c9..e9f1f7d 100644
--- a/emulator/opengl/host/libs/libOpenglRender/WindowSurface.h
+++ b/emulator/opengl/host/libs/libOpenglRender/WindowSurface.h
@@ -38,13 +38,13 @@ public:
EGLSurface getEGLSurface() const { return m_eglSurface; }
void setColorBuffer(ColorBufferPtr p_colorBuffer);
- void flushColorBuffer();
+ bool flushColorBuffer();
void bind(RenderContextPtr p_ctx, SurfaceBindType p_bindType);
private:
WindowSurface();
- void blitToColorBuffer(); // copy pbuffer content with texload and blit
+ bool blitToColorBuffer(); // copy pbuffer content with texload and blit
bool resizePbuffer(unsigned int p_width, unsigned int p_height);
private: