From 338b50c694778f92c641342f4554350acd8c0592 Mon Sep 17 00:00:00 2001 From: njohnson Date: Wed, 10 Sep 2025 21:55:52 -0400 Subject: [PATCH] Fix: Parse Animation Dynamic Indices Index The animation dynamic indices parsing was incorrectly using the stream's `>>` operator. The `XDataStream` class provides `ParseUInt8` for proper parsing of unsigned 8-bit integers, ensuring data integrity. This commit updates the parsing logic to use `ParseUInt8` and include a descriptive string for logging. --- libs/xassets/xanimdynamicindices.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/libs/xassets/xanimdynamicindices.cpp b/libs/xassets/xanimdynamicindices.cpp index 65bc44b..5d58744 100644 --- a/libs/xassets/xanimdynamicindices.cpp +++ b/libs/xassets/xanimdynamicindices.cpp @@ -4,16 +4,12 @@ XAnimDynamicIndices::XAnimDynamicIndices() : XAsset() , mIndices(1) { + SetName("Animation Dynamic Indices"); } -XAnimDynamicIndices::~XAnimDynamicIndices() -{ - -} - -void XAnimDynamicIndices::ParseData(QDataStream *aStream) { +void XAnimDynamicIndices::ParseData(XDataStream *aStream) { if (GetPtr() == -1) { - *aStream >> mIndices[0]; + mIndices[0] = aStream->ParseUInt8(QString("%1 index").arg(GetName())); } }