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

Add XML custom loader overrides #44

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

holywyvern
Copy link

@holywyvern holywyvern commented Dec 28, 2019

This replaces the xmlReaderForFile and xmlFreeTextReader to use instead tmx custom functions instead.

This allows to use PhysFS or other virtual filesystems without much trouble.

I attempted to copy the style used for tmx_img_load_func and tmx_img_free_func as much as I could.

To override this functions and to keep old headers intact, you should require "tmx_xml.h" first.

@holywyvern holywyvern force-pushed the feature-xml_reader_override branch from 4b36973 to aa3aca5 Compare May 10, 2020 14:43
@baylej
Copy link
Owner

baylej commented Jan 14, 2021

Hello, thank you for your contribution.

Sorry for the very late response 😨

I do not want to expose anything from libxml2 in libtmx, so I'm not merging it.
I have an idea for such use cases and I may implement it some day.

@holywyvern
Copy link
Author

Hello, thank you for your contribution.

Sorry for the very late response 😨

I do not want to expose anything from libxml2 in libtmx, so I'm not merging it.
I have an idea for such use cases and I may implement it some day.

Sure, I'll still keep it the branch updated if someone requires this for some reason.

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

2 participants