Less invasive Any CPU implementation inspired by @Chicken-Bones#738
Open
Akarinnnnn wants to merge 122 commits intorlabrecque:masterfrom
Open
Less invasive Any CPU implementation inspired by @Chicken-Bones#738Akarinnnnn wants to merge 122 commits intorlabrecque:masterfrom
Akarinnnnn wants to merge 122 commits intorlabrecque:masterfrom
Conversation
And sorted Skippedlines
But pack aware struct nested in common struct are not in consideration. Write it down here for one day we met some bug related.
A workaround of incorrect codegen of `SteamNetConnectionInfo_t` .
…-conditionalmarshal
…ps://github.com/Akarinnnnn/Steamworks.NET.AnyCPU into anycpu-packable-fix-align-by-conditionalmarshal
…-conditionalmarshal
Puropse is to exclude fake top level structs like CGameID::GameID_t. Maybe just add a table is enough?
Simplify marshal fix #1 's review page, hide these `git subtree` imported commits from by merge them into base branch `anycpu-2`.
…ps://github.com/Akarinnnnn/Steamworks.NET.AnyCPU into anycpu-packable-fix-align-by-conditionalmarshal
Update pre-baked generator resource. Make `SteamMarshallerTable` public. Added `.editorconfig` to enforce style.
Move Any CPU native resolve logic to another file. Minor changes to .editorconfig
Our goal is to fix most callback system related marshal issues. Added a conditional marshalling table to decide which alignment to use at runtime, this should fix most callback-retrieve related issues.. Parameter marshal is done, through it looks ugly. It will be fixed in next upcoming PR. We have a mechanism to find out packsize aware structs now.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Originally posted in #699. Achieved by register a dll resolve hook to
default Assembly Load ContextSteamworks.NET assembly.Dependencies to
static class NativeLibrarymakes this change is not meaningful to any runtimes other than .NET(Core). So these changes are separated into a different folder to keep old version unchanged like .NET Framework 4.0 does.You will see horrible long commit history because the management of
SteamworksParseris switched togit subtree. Which means I importedSteamworksParserfiles and history into this repository. SinceSteamworksParseris always bundled with source code generator, in my practice, I finally decided manage them in monorepo way.