Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dynmem: Add multi-constructor and multi-grow support #53

Open
sriemer opened this issue Jun 10, 2018 · 0 comments
Open

dynmem: Add multi-constructor and multi-grow support #53

sriemer opened this issue Jun 10, 2018 · 0 comments

Comments

@sriemer
Copy link
Member

sriemer commented Jun 10, 2018

The configs of games with too many C++ constructors per class and per object type are too long with too many repetitive entries. What's missing is a possibility to register multiple constructors for the same class. It has been partially done with the grow handling already. The size of the object should also be included and those objects should be dumped in the biggest size.

An example for having too many constructors is the game Dune Legacy.

I imagine this like this:

dynmemstart Unit 256 0x44a04a 0x8
dynmemconst 256 0x44a7ea 0x8
dynmemconst 256 0x449d92 0x8
dynmemconst 256 0x44a53a 0x8
dynmemconst 248 0x449dea 0x8
dynmemconst 248 0x44a58a 0x8
dynmemconst 248 0x449dca 0x8
dynmemconst 248 0x44a56a 0x8
dynmemconst 256 0x449f4a 0x8
dynmemconst 256 0x44a6ea 0x8

HEALTH_VALUE
HOUSE_CHECK

dynmemend

Even when combining only the new mission and the load mission constructor for units and structures, that would save almost half of the config lines already.

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

No branches or pull requests

1 participant