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

Update data.yml, Add AgentInventoryContext::InventoryContextEvent #1275

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

Haselnussbomber
Copy link
Contributor

@Haselnussbomber Haselnussbomber commented Jan 11, 2025

Just for inheritance. Didn't look into what it does.

But I noticed that the ctor for AgentAquariumSetting sets it vtable to +8. Not sure what's going on there, doesn't make sense.

[GenerateInterop]
[Inherits<EventHandler>]
[Inherits<EventHandler>, Inherits<AgentInventoryContext.InventoryContextEvent>]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This seems to have a 3rd inheritance that is calling stuff related to Client::Game::Event::FinalizeSceneTask 2 layers deep in vf1 of said inheritance. Not sure if you want to investigate this before merge.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I know, says so in the comments at the start of the struct.
But I didn't want to look into it, because I didn't want to come up with a name. 😅

Copy link
Collaborator

Choose a reason for hiding this comment

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

ShopSceneTaskScheduler 😛

@wolfcomp wolfcomp merged commit 8c002f0 into aers:main Jan 15, 2025
5 checks passed
@Haselnussbomber Haselnussbomber deleted the add-InventoryContextEvent branch January 15, 2025 10:38
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