Skip to content

Add Rui Asset type#73

Merged
r-ex merged 3 commits into
r-ex:mainfrom
RoyalBlue1:addRuiAssetType
Jan 21, 2026
Merged

Add Rui Asset type#73
r-ex merged 3 commits into
r-ex:mainfrom
RoyalBlue1:addRuiAssetType

Conversation

@RoyalBlue1

Copy link
Copy Markdown
Contributor

Loads rui from RUIP files that are created with SERE
Only support v30 for now

@RoyalBlue1 RoyalBlue1 marked this pull request as ready for review October 22, 2025 13:55
@IJARika

IJARika commented Oct 25, 2025

Copy link
Copy Markdown
Collaborator

can RUIP specific functions and structs be moved to its own header? it keeps the asset files less cluttered, and is more inline with similar files such as MSW.

more of a preference thing, but personally I'd name the asset file 'ui' like RSX does, not a big hitch but yeah.

only main gripes with it currently, I've asked rexx to take a look too.

Move package to own file
Rename rui files to ui
@r-ex r-ex self-requested a review January 14, 2026 00:46

@r-ex r-ex left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

seems good to me! just a few minor formatting changes, otherwise as long as it works i'm happy to merge this. thank you for the implementation!

Comment thread src/RePak.vcxproj.filters Outdated
Comment thread src/assets/ui.cpp Outdated
Comment thread src/assets/ui.cpp Outdated
Comment thread src/logic/pakfile.cpp Outdated
Comment thread src/public/rpak.h Outdated
Comment thread src/public/ui.h Outdated
Comment thread src/public/ui.h Outdated
Comment thread src/public/rui_package.h Outdated
Comment thread src/public/rui_package.h
Comment thread src/public/rpak.h Outdated
@RoyalBlue1

Copy link
Copy Markdown
Contributor Author

Implemented all requested changes

@r-ex r-ex merged commit d6ca9ff into r-ex:main Jan 21, 2026
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.

3 participants