From 6126b3b84a33186ebe3d7230c71bb3ffc963878a Mon Sep 17 00:00:00 2001
From: scrawl <scrawl@baseoftrash.de>
Date: Sat, 25 Oct 2014 17:26:20 +0200
Subject: [PATCH] Videoplayer: wait until we have the first picture before
 returning from playVideo()

---
 .../ogre-ffmpeg-videoplayer/videoplayer.cpp   |  7 ++++
 extern/ogre-ffmpeg-videoplayer/videostate.cpp | 41 ++++++++-----------
 2 files changed, 23 insertions(+), 25 deletions(-)

diff --git a/extern/ogre-ffmpeg-videoplayer/videoplayer.cpp b/extern/ogre-ffmpeg-videoplayer/videoplayer.cpp
index d80449199d..ac759804b4 100644
--- a/extern/ogre-ffmpeg-videoplayer/videoplayer.cpp
+++ b/extern/ogre-ffmpeg-videoplayer/videoplayer.cpp
@@ -31,6 +31,13 @@ void VideoPlayer::playVideo(const std::string &resourceName)
         mState = new VideoState;
         mState->setAudioFactory(mAudioFactory.get());
         mState->init(resourceName);
+
+        // wait until we have the first picture
+        while (mState->video_st && mState->mTexture.isNull())
+        {
+            if (!mState->update())
+                break;
+        }
     }
     catch(std::exception& e) {
         std::cerr<< "Failed to play video: "<<e.what() <<std::endl;
diff --git a/extern/ogre-ffmpeg-videoplayer/videostate.cpp b/extern/ogre-ffmpeg-videoplayer/videostate.cpp
index cc6308b146..9d60cb81c1 100644
--- a/extern/ogre-ffmpeg-videoplayer/videostate.cpp
+++ b/extern/ogre-ffmpeg-videoplayer/videostate.cpp
@@ -207,14 +207,17 @@ void VideoState::video_display(VideoPicture *vp)
 {
     if((*this->video_st)->codec->width != 0 && (*this->video_st)->codec->height != 0)
     {
-
-        if(static_cast<int>(mTexture->getWidth()) != (*this->video_st)->codec->width ||
-           static_cast<int>(mTexture->getHeight()) != (*this->video_st)->codec->height)
+        if (mTexture.isNull())
         {
-            mTexture->unload();
-            mTexture->setWidth((*this->video_st)->codec->width);
-            mTexture->setHeight((*this->video_st)->codec->height);
-            mTexture->createInternalResources();
+            static int i = 0;
+            mTexture = Ogre::TextureManager::getSingleton().createManual(
+                            "ffmpeg/VideoTexture" + Ogre::StringConverter::toString(++i),
+                                    Ogre::ResourceGroupManager::DEFAULT_RESOURCE_GROUP_NAME,
+                                    Ogre::TEX_TYPE_2D,
+                                    (*this->video_st)->codec->width, (*this->video_st)->codec->height,
+                                    0,
+                                    Ogre::PF_BYTE_RGBA,
+                                    Ogre::TU_DYNAMIC_WRITE_ONLY_DISCARDABLE);
         }
         Ogre::PixelBox pb((*this->video_st)->codec->width, (*this->video_st)->codec->height, 1, Ogre::PF_BYTE_RGBA, &vp->data[0]);
         Ogre::HardwarePixelBufferSharedPtr buffer = mTexture->getBuffer();
@@ -657,24 +660,6 @@ void VideoState::init(const std::string& resourceName)
     if(video_index >= 0)
     {
         this->stream_open(video_index, this->format_ctx);
-
-        int width = (*this->video_st)->codec->width;
-        int height = (*this->video_st)->codec->height;
-        static int i = 0;
-        this->mTexture = Ogre::TextureManager::getSingleton().createManual(
-                        "ffmpeg/VideoTexture" + Ogre::StringConverter::toString(++i),
-                                Ogre::ResourceGroupManager::DEFAULT_RESOURCE_GROUP_NAME,
-                                Ogre::TEX_TYPE_2D,
-                                width, height,
-                                0,
-                                Ogre::PF_BYTE_RGBA,
-                                Ogre::TU_DYNAMIC_WRITE_ONLY_DISCARDABLE);
-
-        // initialize to (0,0,0,0)
-        std::vector<Ogre::uint32> buffer;
-        buffer.resize(width * height, 0);
-        Ogre::PixelBox pb(width, height, 1, Ogre::PF_BYTE_RGBA, &buffer[0]);
-        this->mTexture->getBuffer()->blitFromMemory(pb);
     }
 
 
@@ -724,6 +709,12 @@ void VideoState::deinit()
         }
         avformat_close_input(&this->format_ctx);
     }
+
+    if (!mTexture.isNull())
+    {
+        Ogre::TextureManager::getSingleton().remove(mTexture->getName());
+        mTexture.setNull();
+    }
 }
 
 double VideoState::get_external_clock()