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

Allow loading a Tileset without a Map #128

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

Conversation

mpartel
Copy link
Contributor

@mpartel mpartel commented Jun 20, 2023

Useful e.g. for generating maps in code.

Includes some cleanup commits.

Not super extensively tested, since I don't have a large project using this yet.

…ing directory, and reduced duplicate code in load()
The warning was: ISO C++20 considers use of overloaded operator '==' (with operand types 'tmx::Colour' and 'tmx::Colour') to be ambiguous despite there being a unique best viable function [-Wambiguous-reversed-operator]
@fallahn
Copy link
Owner

fallahn commented Jun 21, 2023

Hi! Thanks for this.
I'm a little reluctant to add this right now if you say it's not fully tested - however please do keep this up to date with any changes so I can re-evaluate it down the line! 😁 If the clean up commits are important (such as the Colour operator) you can always move them to a separate PR.

@mpartel
Copy link
Contributor Author

mpartel commented Jun 21, 2023

I'm actually not sure where the bar should be for testing 🤔
The change works for me, and it doesn't break the included examples 🤷‍♂️
If it also works with your games, and passes your review, perhaps that's good enough?

Anyway, no pressure, I don't mind using a forked version in my project.

@fallahn
Copy link
Owner

fallahn commented Jun 21, 2023

Sure! I'll give it a go when I have a moment - if nothing is broken then I'm sure it'll be OK to merge 😁

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