Skip to content

Conversation

@martonmiklos
Copy link
Contributor

Describe your changes

This PR adds support for parsing object pools containing scaled graphics

Fixes # (issue)

How has this been tested?

NX9 VT6 test implement contains a scaled graphics object.

@martonmiklos martonmiklos requested a review from GwnDaan November 30, 2025 22:16
@martonmiklos martonmiklos force-pushed the martonmiklos/add_support_for_scaled_graphics branch from 711dc02 to 66d0743 Compare November 30, 2025 22:26
@martonmiklos martonmiklos changed the title Add support for parsing Scaleable Graphics objects from the IOP pool Add support for parsing Scaled Graphics objects from the IOP pool Dec 1, 2025
@martonmiklos martonmiklos force-pushed the martonmiklos/add_support_for_scaled_graphics branch from 98ca642 to 1d328a8 Compare December 1, 2025 10:48
@martonmiklos martonmiklos force-pushed the martonmiklos/add_support_for_scaled_graphics branch from 1d328a8 to d95945d Compare December 1, 2025 11:02
@martonmiklos martonmiklos force-pushed the martonmiklos/add_support_for_scaled_graphics branch from 321fe40 to 32007a5 Compare December 1, 2025 21:15
@martonmiklos martonmiklos force-pushed the martonmiklos/add_support_for_scaled_graphics branch from 32007a5 to 0e78ef6 Compare December 1, 2025 21:16
@martonmiklos martonmiklos requested a review from GwnDaan December 1, 2025 21:19
@sonarqubecloud
Copy link

Comment on lines +2855 to +2860
if (iopLength >= sizeOfMacros)
{
for (std::uint_fast8_t i = 0; i < numberOfMacrosToFollow; i++)
{
// If the first byte is 255, then more bytes are used! 4.6.22.3
if (iopData[0] == static_cast<std::uint8_t>(EventID::UseExtendedMacroReference))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should use the function you made in #630

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants