AngelScript: Adding support for displaying Hammer objects in the reference API. #92
+103
−76
Conversation
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
…ki or the Wiki docs.
However, still has issues with some types or enums like "Material" where it's a opaque class for Hammer, but for the game it has a lot more to the class but the Hammer version completly overrides the game version.
OrsellGit
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The current implementation of the AngelScript (AS) wiki dump parser parses the server and client dump files. However, since there is currently no real difference between the two, all functions, enums, and types are marked as "Server & Client". The only AS implementation difference in Strata Source is with Hammer, which includes various class types and enums that are NOT in the game. Ex.
BoundBox,RenderModeRight now, as of 1/4/26, the parser has an issue if a type or enum exists in BOTH the game and Hammer. An example is
Material, whereMaterialexists to a far greater extent than Hammer, with several functions in the class definition. However, for Hammer, it's an empty shell. The "mergeDump" function currently doesn't handle a type already being defined and instead overwrites what's already there with the next dump to be merged. It wasn't an issue with the client & server ones since they were currently the same, but not the Hammer one.The goal of this PR is to rewrite/update the AS wiki parser to support the Hammer dump file. This PR might evolve into rewriting the entire dump parsing scene, so that all scripting languages can be parsed from using one parsing file and formatted into the same format for wiki-wide consistency.
TODO: