libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2025-04-03 07:11:49 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

TWINE: Fade-in characters starting from the right when playing in Hebrew.

Browse Source 
Hebrew translation use ISO-8859-8 in visual ordering. This is nasty but we can't
change it. Instead do the best with what we have. One of problems is that
characters fade in in the order that is reverse to reading order. This patch
improves this and sets a groundwork for other RTL languages, hopefully in
logical ordering.
This commit is contained in:
Vladimir Serbinenko 2023-05-12 03:42:38 +02:00 committed by Martin Gerhardy
parent a40cf3a442
commit ed88fc457d
2 changed files with 71 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

88
engines/twine/text.cpp
View File

@ -54,6 +54,7 @@ static const int kLBASJISCharHeight = 24;
Text::Text(TwinEEngine *engine) : _engine(engine) {
Common::fill(&_currMenuTextBuffer[0], &_currMenuTextBuffer[256], 0);
_isShiftJIS = _engine->getGameLang() == Common::Language::JA_JPN;
_isVisualRTL = _engine->getGameLang() == Common::Language::HE_ISR;
}
Text::~Text() {
@ -374,9 +375,10 @@ void Text::initText(TextId index) {
_hasValidTextHandle = true;
_dialTextBoxCurrentLine = 0;
_progressiveTextBuffer[0] = '\0';
_progressiveTextBuffer[0].chr = '\0';
_progressiveTextBuffer[0].x = 0;
_fadeInCharactersPos = 0;
_dialTextXPos = _dialTextBox.left + PADDING;
_dialTextBaseXPos = _dialTextBox.left + PADDING;
_dialTextYPos = _dialTextBox.top + PADDING;
_currentTextPosition = _currDialTextPtr;
@ -388,19 +390,23 @@ void Text::initText(TextId index) {
}
void Text::initProgressiveTextBuffer() {
Common::fill(&_progressiveTextBuffer[0], &_progressiveTextBuffer[sizeof(_progressiveTextBuffer)], ' ');
for (uint i = 0; i < ARRAYSIZE(_progressiveTextBuffer); i++) {
_progressiveTextBuffer[i].chr = ' ';
_progressiveTextBuffer[i].x = (_dialCharSpace + 1) * i;
}
// the end of the buffer defines how fast the next page is shown - as the
// whitespaces are handled in the fade in process, too. But we need at least 32 chars,
// to completly fade in the last characters of a full page (see TEXT_MAX_FADE_IN_CHR)
_progressiveTextBuffer[sizeof(_progressiveTextBuffer) - 1] = '\0';
_progressiveTextBuffer[sizeof(_progressiveTextBuffer) - 1].chr = '\0';
_progressiveTextBuffer[sizeof(_progressiveTextBuffer) - 1].x = 0;
_progressiveTextBufferPtr = _progressiveTextBuffer;
_dialTextBoxCurrentLine = 0;
}
void Text::fillFadeInBuffer(int16 x, int16 y, int16 chr) {
void Text::fillFadeInBuffer(int16 baseX, int16 y, const LineCharacter &chr) {
if (_fadeInCharactersPos < TEXT_MAX_FADE_IN_CHR) {
_fadeInCharacters[_fadeInCharactersPos].chr = chr;
_fadeInCharacters[_fadeInCharactersPos].x = x;
_fadeInCharacters[_fadeInCharactersPos].chr = chr.chr;
_fadeInCharacters[_fadeInCharactersPos].x = baseX + chr.x;
_fadeInCharacters[_fadeInCharactersPos].y = y;
_fadeInCharactersPos++;
return;
@ -412,8 +418,8 @@ void Text::fillFadeInBuffer(int16 x, int16 y, int16 chr) {
_fadeInCharacters[var2] = _fadeInCharacters[var1];
counter2++;
}
_fadeInCharacters[TEXT_MAX_FADE_IN_CHR - 1].chr = chr;
_fadeInCharacters[TEXT_MAX_FADE_IN_CHR - 1].x = x;
_fadeInCharacters[TEXT_MAX_FADE_IN_CHR - 1].chr = chr.chr;
_fadeInCharacters[TEXT_MAX_FADE_IN_CHR - 1].x = baseX + chr.x;
_fadeInCharacters[TEXT_MAX_FADE_IN_CHR - 1].y = y;
}
@ -436,6 +442,26 @@ Text::WordSize Text::getWordSize(const char *completeText, char *wordBuf, int32
return size;
}
void Text::appendProgressiveTextBuffer(const char *s, int &x, uint &i) {
while (1) {
if (i >= ARRAYSIZE(_progressiveTextBuffer) - 1)
return;
uint16 chr = getNextChar(s);
if (!chr)
return;
_progressiveTextBuffer[i].chr = chr;
_progressiveTextBuffer[i].x = x;
i++;
if (chr == ' ') {
x += _dialCharSpace + 1;
} else {
x += getCharWidth(chr) + 2;
}
}
}
void Text::processTextLine() {
const char *buffer = _currentTextPosition;
_dialCharSpace = 7;
@ -443,7 +469,10 @@ void Text::processTextLine() {
int32 lineBreakX = 0;
int32 spaceCharCount = 0;
_progressiveTextBuffer[0] = '\0';
int x = 0;
uint i = 0;
_progressiveTextBuffer[0].chr = '\0';
_progressiveTextBuffer[0].x = '\0';
for (;;) {
if (*buffer == ' ') {
@ -473,8 +502,7 @@ void Text::processTextLine() {
buffer++;
if (lineBreakX == 0) {
lineBreakX = 7;
*(_progressiveTextBuffer + 0) = ' ';
*(_progressiveTextBuffer + 1) = '\0';
appendProgressiveTextBuffer(" ", x, i);
}
// new page?
if (wordBuf[1] == 'P') {
@ -486,8 +514,8 @@ void Text::processTextLine() {
buffer += wordSize.inChar;
_currentTextPosition = buffer;
strncat(_progressiveTextBuffer, wordBuf, sizeof(_progressiveTextBuffer) - strlen(_progressiveTextBuffer) - 1);
strncat(_progressiveTextBuffer, " ", sizeof(_progressiveTextBuffer) - strlen(_progressiveTextBuffer) - 1);
appendProgressiveTextBuffer(wordBuf, x, i);
appendProgressiveTextBuffer(" ", x, i);
spaceCharCount++;
lineBreakX += wordSize.inPixel + _dialCharSpace;
@ -498,6 +526,17 @@ void Text::processTextLine() {
break;
}
_progressiveTextBuffer[i].chr = 0;
_progressiveTextBuffer[i].x = 0;
if (_isVisualRTL) {
for (uint j = 0; j < i / 2; j++) {
LineCharacter t = _progressiveTextBuffer[j];
_progressiveTextBuffer[j] = _progressiveTextBuffer[i - j - 1];
_progressiveTextBuffer[i - j - 1] = t;
}
}
if (spaceCharCount > 0) {
spaceCharCount--;
}
@ -599,27 +638,20 @@ ProgressiveTextState Text::updateProgressiveText() { // NextDialCar
return ProgressiveTextState::End;
}
if (*_progressiveTextBufferPtr == '\0') {
if (_progressiveTextBufferPtr->chr == '\0') {
initProgressiveTextBuffer();
processTextLine();
initDialogueBox();
_dialTextXPos = _dialTextBox.left + PADDING;
_dialTextBaseXPos = _dialTextBox.left + PADDING;
_dialTextYPos = _dialTextBox.top + PADDING;
}
const uint16 currentChar = getNextChar(_progressiveTextBufferPtr);
assert(currentChar != '\0');
fillFadeInBuffer(_dialTextXPos, _dialTextYPos, currentChar);
LineCharacter currentChar = *_progressiveTextBufferPtr++;
assert(currentChar.chr != '\0');
fillFadeInBuffer(_dialTextBaseXPos, _dialTextYPos, currentChar);
fadeInCharacters(_fadeInCharactersPos, _dialTextStartColor);
const int8 charWidth = getCharWidth(currentChar);
if (currentChar == ' ') {
_dialTextXPos += _dialCharSpace + 1;
} else {
_dialTextXPos += charWidth + 2;
}
// reaching 0-byte means a new line - as we are fading in per line
if (*_progressiveTextBufferPtr != '\0') {
if (_progressiveTextBufferPtr->chr != '\0') {
return ProgressiveTextState::ContinueRunning;
}
@ -633,7 +665,7 @@ ProgressiveTextState Text::updateProgressiveText() { // NextDialCar
_dialTextBoxCurrentLine++;
_dialTextYPos += lineHeight;
_dialTextXPos = _dialTextBox.left + PADDING;
_dialTextBaseXPos = _dialTextBox.left + PADDING;
if (_dialTextBoxCurrentLine >= _dialTextBoxLines) {
renderContinueReadingTriangle();

15
engines/twine/text.h
View File

@ -65,15 +65,20 @@ private:
int32 inChar = 0;
int32 inPixel = 0;
};
struct LineCharacter {
int16 chr = 0;
int16 x = 0;
};
WordSize getWordSize(const char *completeText, char *wordBuf, int32 wordBufSize);
uint16 getNextChar(const char *&dialogue);
void processTextLine();
void appendProgressiveTextBuffer(const char *s, int &x, uint &i);
// draw next page arrow polygon
void renderContinueReadingTriangle();
/**
* @see fadeInCharacters
*/
void fillFadeInBuffer(int16 x, int16 y, int16 chr);
void fillFadeInBuffer(int16 baseX, int16 y, const LineCharacter &chr);
/**
* Blend in characters for a text scrolling in
*
@ -84,14 +89,14 @@ private:
TextBankId _currentBankIdx = TextBankId::None;
char _progressiveTextBuffer[256] {'\0'};
LineCharacter _progressiveTextBuffer[256];
const char *_currentTextPosition = nullptr;
int32 _dialTextXPos = 0;
int32 _dialTextBaseXPos = 0;
int32 _dialTextYPos = 0;
/** Current position of in the buffer of characters that are currently faded in */
const char *_progressiveTextBufferPtr = nullptr;
const LineCharacter *_progressiveTextBufferPtr = nullptr;
int32 _dialTextBoxCurrentLine = 0;
struct BlendInCharacter {
@ -137,6 +142,7 @@ private:
int32 _dialTextBoxMaxX = 0; // dialogueBoxParam2
bool _isShiftJIS = false;
bool _isVisualRTL = false;
bool displayText(TextId index, bool showText, bool playVox, bool loop);
public:
@ -193,6 +199,7 @@ public:
void initInventoryDialogueBox();
void initText(TextId index);
void initLine();
void initInventoryText(InventoryItems index);
void initItemFoundText(InventoryItems index);
void fadeInRemainingChars();