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

Updated Mangos.Extractor to support WoTLK and Cata. Credit to Brotalnia. #71

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

billy1arm
Copy link
Contributor

@billy1arm billy1arm commented Jan 24, 2023

This change is Reviewable

@billy1arm billy1arm changed the title Unified extractor Updated Mangos.Extractor to support WoTLK and Cata. Credit to Brotalnia. Jan 24, 2023
@billy1arm
Copy link
Contributor Author

The following was in @Daribon original PR:

This adds support to WoTLK and Cataclysm to Mangos.Extractor. This means that the extractor supports all versions between 0.5.3 alpha to 4.3.4 cataclysm after it is applied.

Credit goes out to Brotalnia for most of the code in this commit. Brotalnia gave me permission to publish his code. I just updated the code to match the formatting and coding style of the latest revision, since it was based on an older revision.

The patch also changes the following:

Updatefields output has been changed from C# to C++. Since the latter is widely more used in WoW emulation.
Removed patch and build from filename output, because it already includes it within the output text file.
Removed auto generated time and date from output text file, since its irrelevant.
Updatefield extractor is now an universal extractor for all versions with updatefield names in .exe. It will not run on any MoP patch or above, because the client stopped having updatefield names visible in MoP+.
Updated the extractors for opcodes, SpellFailedReasons and ChatTypes.
As for the first lines in the commit with no changes, I think someone prior to this commit mixed up windows and unix line endings. I tried to filter it out as best as I could on commit.

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