From 798af8959a36192f0e1a48066c309f08eb774bc0 Mon Sep 17 00:00:00 2001
From: Kirill Smelkov <kirr@navytux.spb.ru>
Date: Sat, 16 Jun 2012 14:12:48 +0400
Subject: [PATCH] view: Check return value of SDL_LockYUVOverlay()

Just in case. NOTE SDL_UnlockYUVOverlay() does not need checking - it
returns void.

Signed-off-by: Kirill Smelkov <kirr@navytux.spb.ru>
---
 view.cpp | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/view.cpp b/view.cpp
index 8d59366..b933703 100644
--- a/view.cpp
+++ b/view.cpp
@@ -49,6 +49,20 @@ View::~View()
 }
 
 
+/* non-forgiving helper for SDL_LockYUVOverlay();
+ *
+ * NOTE SDL_UnlockYUVOverlay returns void, so no need to check.
+ */
+static void SDLU_XLockYUVOverlay(SDL_Overlay *overlay)
+{
+    int err;
+
+    err = SDL_LockYUVOverlay(overlay);
+    if (err)
+        die("view: SDL_LockYUVOverlay() failed");
+}
+
+
 void View::display_frame(const Frame *f)
 {
 #if 0
@@ -81,7 +95,7 @@ void View::display_frame(const Frame *f)
     }
 
 
-    SDL_LockYUVOverlay(overlay);
+    SDLU_XLockYUVOverlay(overlay);
 
     switch (f->pixfmt_4cc) {
         case MKTAG32('Y','U','Y','V'):
-- 
2.11.4.GIT