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

Add SetupSteamApi to Framework #1247

Merged
merged 2 commits into from
Dec 10, 2024
Merged

Add SetupSteamApi to Framework #1247

merged 2 commits into from
Dec 10, 2024

Conversation

KazWolfe
Copy link
Contributor

@KazWolfe KazWolfe commented Dec 9, 2024

Needed so that we can manually call SetupSteamApi() from Dalamud/plugins.

The game will automatically tear down the Steam API when shutting itself down, so long as the SteamApi pointer is set (see Client::System::Framework::Framework_TeardownSteamApi, inlined in Framework#Free.

This method must only be called once per session, so callers need to verify first that IsSteamApiInitialized returns false, and that boot/steam_api64.dll is present. If a Steam API is already present, re-calling this method will reinitialize the API, leading to potentially problematic edge cases.

Likewise, if this method is called when boot/steam_api64.dll is not present, it will set the variable Framework#IsSteamGame to true. However, this does not appear to actually do anything, so perhaps this is safe.

Co-authored-by: Haselnussbomber <mail@haselnussbomber.de>
Copy link
Contributor

github-actions bot commented Dec 9, 2024

Breaking Changes

Type exists in left but not in right

FFXIVClientStructs.FFXIV.Component.GUI: 1

AtkEventData: 5

  • AtkMouseData
  • AtkInputData
  • AtkFocusData
  • AtkDragDropData
  • AtkAddonControlData

Member exists in left but not in right

FFXIVClientStructs.FFXIV.Component.GUI: 3

AtkComponentTextInput: 1

  • ushort InputSanitizationFlags

AtkEventType: 2

  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventType MouseWheel
  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventType SliderValueUpdate

AtkEventData: 5

  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventData.AtkMouseData MouseData
  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventData.AtkInputData InputData
  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventData.AtkFocusData FocusData
  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventData.AtkDragDropData DragDropData
  • FFXIVClientStructs.FFXIV.Component.GUI.AtkEventData.AtkAddonControlData AddonControlData

@github-actions github-actions bot added the breaking change PR contains breaking changes and wont be merged before a new patch label Dec 9, 2024
@wolfcomp wolfcomp merged commit 02dcf10 into aers:main Dec 10, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change PR contains breaking changes and wont be merged before a new patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants