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

Deprecate or adapt Masking? #49

Open
viking-sudo-rm opened this issue Sep 5, 2018 · 3 comments
Open

Deprecate or adapt Masking? #49

viking-sudo-rm opened this issue Sep 5, 2018 · 3 comments
Labels
low priority Nice to have, but not needed. question

Comments

@viking-sudo-rm
Copy link
Member

Masking techniques were used in the old game to blend a continuous surface between bordering tile regions. This is probably no longer relevant, so it should be removed from the code base.

@viking-sudo-rm viking-sudo-rm added question low priority Nice to have, but not needed. labels Sep 5, 2018
@viking-sudo-rm
Copy link
Member Author

There is a lot of logic for masking in TileLayer.java.

@viking-sudo-rm
Copy link
Member Author

Status: masking logic should be removed wherever possible. This will probably lead to big improvements in performance.

@viking-sudo-rm
Copy link
Member Author

Remove Masking, but transfer some useful logic from this class into something else.

In particular, we need the logic for connectivity between grid nodes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Nice to have, but not needed. question
Projects
None yet
Development

No branches or pull requests

1 participant