From 0a9ad229a89a04b2fe437597584684f43479ca14 Mon Sep 17 00:00:00 2001
From: Peter Kohaut
Date: Tue, 22 Jan 2019 18:59:42 +0100
Subject: BLADERUNNER: Fixed outtake player

In some scenes, background video got corrupted as outtake was rendered
into a wrong surface.
---
 engines/bladerunner/outtake.cpp | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

(limited to 'engines/bladerunner/outtake.cpp')

diff --git a/engines/bladerunner/outtake.cpp b/engines/bladerunner/outtake.cpp
index 4698a83399..fc610b20e9 100644
--- a/engines/bladerunner/outtake.cpp
+++ b/engines/bladerunner/outtake.cpp
@@ -32,6 +32,15 @@
 
 namespace BladeRunner {
 
+OuttakePlayer::OuttakePlayer(BladeRunnerEngine *vm) {
+	_vm = vm;
+	_surfaceVideo.create(_vm->_surfaceBack.w, _vm->_surfaceBack.h, createRGB555());
+}
+
+OuttakePlayer::~OuttakePlayer() {
+	_surfaceVideo.free();
+}
+
 void OuttakePlayer::play(const Common::String &name, bool noLocalization, int container) {
 	if (container > 0) {
 		debug("OuttakePlayer::play TODO");
@@ -45,7 +54,7 @@ void OuttakePlayer::play(const Common::String &name, bool noLocalization, int co
 	Common::String resNameNoVQASuffix = resName;
 	resName = resName + ".VQA";
 
-	VQAPlayer vqa_player(_vm, &_vm->_surfaceBack, resName); // surfaceBack is needed here for subtitles rendering properly, original was _surfaceFront here
+	VQAPlayer vqa_player(_vm, &_surfaceVideo, resName); // in original game _surfaceFront is used here, but for proper subtitles rendering we need separate surface
 
 	vqa_player.open();
 
@@ -57,9 +66,10 @@ void OuttakePlayer::play(const Common::String &name, bool noLocalization, int co
 				return;
 
 		int frame = vqa_player.update();
-		blit(_vm->_surfaceBack, _vm->_surfaceFront); // This helps to make subtitles disappear properly, if the video is rendered in surface back and then pushed to the front surface
-		if (frame == -3)
+		blit(_surfaceVideo, _vm->_surfaceFront); // This helps to make subtitles disappear properly, if the video is rendered in separate surface and then pushed to the front surface
+		if (frame == -3) { // end of video
 			break;
+		}
 
 		if (frame >= 0) {
 			_vm->_subtitles->getOuttakeSubsText(resNameNoVQASuffix, frame);
@@ -69,6 +79,8 @@ void OuttakePlayer::play(const Common::String &name, bool noLocalization, int co
 
 		_vm->_system->delayMillis(10);
 	}
+
+	// TODO: scene video is played fast after end of outtake, fix it (visible in tb07)
 }
 
 } // End of namespace BladeRunner
-- 
cgit v1.2.3