From 78fd8d39c500cb85fdf5766ce1d836816264fad6 Mon Sep 17 00:00:00 2001 From: Colin Snover Date: Sat, 25 Feb 2017 11:32:24 -0600 Subject: SCI32: Disable brightness boost when black-lined video is disabled The boost is intended to compensate for the darkness caused by the black lines, so should not be applied when there are no black lines. This fixes too-bright videos in at least LSL7. --- engines/sci/graphics/video32.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'engines/sci') diff --git a/engines/sci/graphics/video32.cpp b/engines/sci/graphics/video32.cpp index 0e8ac8af41..1759e8e722 100644 --- a/engines/sci/graphics/video32.cpp +++ b/engines/sci/graphics/video32.cpp @@ -590,7 +590,10 @@ void VMDPlayer::init(const int16 x, const int16 y, const PlayFlags flags, const _y = y; _doublePixels = flags & kPlayFlagDoublePixels; _blackLines = ConfMan.getBool("enable_black_lined_video") && (flags & kPlayFlagBlackLines); - _boostPercent = 100 + (flags & kPlayFlagBoost ? boostPercent : 0); + // If ScummVM has been configured to disable black lines on video playback, + // the boosts need to be ignored too or else the brightness of the video + // will be too high + _boostPercent = 100 + (_blackLines && (flags & kPlayFlagBoost) ? boostPercent : 0); _boostStartColor = CLIP(boostStartColor, 0, 255); _boostEndColor = CLIP(boostEndColor, 0, 255); _leaveScreenBlack = flags & kPlayFlagLeaveScreenBlack; -- cgit v1.2.3