Skip to content

feat: add addon map ents dumper to T6#842

Open
meowlyny wants to merge 5 commits into
Laupetin:mainfrom
meowlyny:addon-ents-dumper
Open

feat: add addon map ents dumper to T6#842
meowlyny wants to merge 5 commits into
Laupetin:mainfrom
meowlyny:addon-ents-dumper

Conversation

@meowlyny

Copy link
Copy Markdown
Contributor

Adds the abillity to dump AddonMapEnts assets from T6.
Zombie gametype ents and spec ops (Strike Force) ents from Campaign are now dumpable

@@ -0,0 +1,21 @@
#define NOMINMAX

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.

That should not be necessary. The min and max macros are usually defined by the windows.h header, which should not be included anywhere here.

@meowlyny meowlyny Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hey so I based this off the IW5 one, so it should also be removed from the others too

return;

auto& stream = *assetFile;
stream.write(addonMapEnts->entityString, std::max(addonMapEnts->numEntityChars - 1, 0));

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.

The big issue is that this asset is more data than just the entity string.

e.g.
Image

Just dumping the entity string is not necessarly very helpful as you cannot reconstruct the asset from just that.
So it's not possible to create a new addonmapents asset from disk like that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Since Plutonium could load them from disk (assuming it's only entity string), I thought it would be useful for modders to be able to dump it alongside with map ents. I am aware Plutonium can dump them with dump_assets dvar

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.

2 participants