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

patterns: Added Kaydara FBX Binary format #271

Closed
wants to merge 5 commits into from

Conversation

Hikodroid
Copy link
Contributor

This hexpat parses a binary FBX file to give a high-level overview of its contents.

@Hikodroid
Copy link
Contributor Author

Fixed a bug and added support for decompressing arrays. The pattern should be feature-complete now.

@Hikodroid Hikodroid closed this Jul 19, 2024
@C3pa
Copy link
Contributor

C3pa commented Jul 23, 2024

Why did you decide to close the PR? WevWolv might have just missed it.

@WerWolv
Copy link
Owner

WerWolv commented Jul 23, 2024

Might have been a mistake yeah. I'm absolutely open for merging this

@WerWolv
Copy link
Owner

WerWolv commented Jul 23, 2024

Ah the original repo was deleted. I'll just merge it manually then. Thanks a lot for the effort!

@Hikodroid
Copy link
Contributor Author

nestedList seems to have some issues that I could not fix. I discovered late that the pattern fails to cover some FBX files entirely and I have not been able to find the cause of it, but it seems to be related to the way I handled NodeRecord subtrees. Therefore I closed the PR because I did not want to submit something that is broken. Sorry for the confusion.

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.

None yet

3 participants