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

SylkParser bug fixes #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

speige
Copy link
Contributor

@speige speige commented Dec 8, 2024

Make SylkParser more robust to handle malformed files & ignore useless tags, like formatting.

@Drake53
Copy link
Owner

Drake53 commented Dec 19, 2024

Do you have example .slk files which use these tags? I know the implementation is incomplete because I only intended to load the .slk files from wc3, which only use the ID, B, C, and E tags.

@speige
Copy link
Contributor Author

speige commented Dec 19, 2024

SLK Files.zip
These are all files that are part of the base game extracted from the casc. They fail with the old SylkParser but work with the new version.

I can also get you examples from custom maps, if needed.

@speige speige force-pushed the SylkParser_bug_fixes branch 6 times, most recently from f21bf4e to 9711ef3 Compare January 5, 2025 11:55
@speige speige force-pushed the SylkParser_bug_fixes branch from 9711ef3 to 7110797 Compare January 5, 2025 12:05
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