summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorrobojumper <robojumper@gmail.com>2025-11-22 22:24:38 +0100
committerrobojumper <robojumper@gmail.com>2025-11-22 22:24:38 +0100
commitab83c7628f2f1bb1d8842cfee839338a8ccc86d9 (patch)
tree51c201ce015867b337bcf729d090c0a15a06ec60 /src
parent54fc6c4a588d0642afaaabb7e0f26f27231c3af5 (diff)
Use a common constant for msg window character buffer size
Diffstat (limited to 'src')
-rw-r--r--src/d/d_textwindow_unk.cpp4
-rw-r--r--src/d/lyt/msg_window/d_lyt_msg_window_sword.cpp14
-rw-r--r--src/d/lyt/msg_window/d_lyt_simple_window.cpp9
3 files changed, 13 insertions, 14 deletions
diff --git a/src/d/d_textwindow_unk.cpp b/src/d/d_textwindow_unk.cpp
index 22782a5d..063df1b9 100644
--- a/src/d/d_textwindow_unk.cpp
+++ b/src/d/d_textwindow_unk.cpp
@@ -70,14 +70,14 @@ bool TextWindowUnk::checkEndReached() const {
}
void MsgWindowBlurRelated::build() {
- for (int i = 0; i < 1023; i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
mData[i].reset();
}
}
void MsgWindowBlurRelated::drawTextBlur(dLytTextLight *thing, mVec2_c size, u8 alpha) {
s8 t = -1;
- for (int i = 0; i < 1023; i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (mData[i].displayTimerMaybe >= 0) {
if (t == -1) {
mData[i].displayTimerMaybe++;
diff --git a/src/d/lyt/msg_window/d_lyt_msg_window_sword.cpp b/src/d/lyt/msg_window/d_lyt_msg_window_sword.cpp
index 27e725f8..16426fc2 100644
--- a/src/d/lyt/msg_window/d_lyt_msg_window_sword.cpp
+++ b/src/d/lyt/msg_window/d_lyt_msg_window_sword.cpp
@@ -4,6 +4,7 @@
#include "d/d_gfx.h"
#include "d/d_tag_processor.h"
#include "d/d_lyt_hio.h"
+#include "d/d_textwindow_unk.h"
#include "d/lyt/d2d.h"
#include "d/lyt/d_textbox.h"
#include "m/m_vec.h"
@@ -96,7 +97,7 @@ bool dLytMsgWindowSword_c::build(d2d::ResAccIf_c *resAcc1, d2d::ResAccIf_c *resA
mSwordText.build(resAcc, mSwordType);
- for (int i = 0; i < 1023; i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
mCharacterData[i].reset();
mTextBuf[i] = L'\0';
}
@@ -265,7 +266,7 @@ bool dLytMsgWindowSword_c::execute() {
void dLytMsgWindowSword_c::draw() {
mLyt.draw();
- for (int i = 0; i < 1023; i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (mCharacterData[i].displayTimerMaybe >= 0) {
if (!mHasDrawnThisTick) {
mCharacterData[i].displayTimerMaybe += 1;
@@ -313,15 +314,14 @@ bool dLytMsgWindowSword_c::isDoneClosing() const {
}
bool dLytMsgWindowSword_c::setText(const wchar_t *text) {
- int foundIdx = 1023;
- for (int i = 0; i < 1023; i++) {
+ int foundIdx = TextWindowUnk::BUF_SIZE;
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (mCharacterData[i].displayTimerMaybe >= 0 && foundIdx > mCharacterData[i].field_0x0C) {
foundIdx = mCharacterData[i].field_0x0C;
}
}
- // How is text bounds-checked here????
- for (int i = 0; i < 1023; i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (i < foundIdx) {
mTextBuf[i] = text[i];
} else {
@@ -333,5 +333,5 @@ bool dLytMsgWindowSword_c::setText(const wchar_t *text) {
mpTextboxes[1]->SetTagProcessor(mpTagProcessor);
mpTextboxes[0]->setTextWithTextProcessor(mTextBuf, mpTagProcessor, 0);
mpTextboxes[1]->setTextWithTextProcessor(mTextBuf, mpTagProcessor, 0);
- return foundIdx == 1023;
+ return foundIdx == TextWindowUnk::BUF_SIZE;
}
diff --git a/src/d/lyt/msg_window/d_lyt_simple_window.cpp b/src/d/lyt/msg_window/d_lyt_simple_window.cpp
index 67cb522c..9991aae7 100644
--- a/src/d/lyt/msg_window/d_lyt_simple_window.cpp
+++ b/src/d/lyt/msg_window/d_lyt_simple_window.cpp
@@ -280,7 +280,7 @@ bool dLytSimpleWindowSword_c::build(d2d::ResAccIf_c *resAcc, dTagProcessor_c *ta
mpTextboxes[1]->setIsShadow(1);
mpTagProcessor = tagProcessor;
- for (int i = 0; i < (int)ARRAY_LENGTH(mCharData); i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
mCharData[i].reset();
mTextBuf[i] = L'\0';
}
@@ -373,15 +373,14 @@ bool dLytSimpleWindowSword_c::isOut() {
}
bool dLytSimpleWindowSword_c::setText(const wchar_t *text) {
- int foundIdx = ARRAY_LENGTH(mCharData);
- for (int i = 0; i < (int)ARRAY_LENGTH(mCharData); i++) {
+ int foundIdx = TextWindowUnk::BUF_SIZE;
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (mCharData[i].displayTimerMaybe >= 0 && foundIdx > mCharData[i].field_0x0C) {
foundIdx = mCharData[i].field_0x0C;
}
}
- // How is text bounds-checked here????
- for (int i = 0; i < (int)ARRAY_LENGTH(mCharData); i++) {
+ for (int i = 0; i < TextWindowUnk::BUF_SIZE; i++) {
if (i < foundIdx) {
mTextBuf[i] = text[i];
} else {