Skip to content

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 holywyvern force-pushed the feature-xml_reader_override branch from e24b546 to 4c2a442 Compare January 15, 2021 17:32
@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.

@baylej baylej force-pushed the master branch 12 times, most recently from cff450c to 3032fe4 Compare March 11, 2024 21:22
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