nguillaumin / slick2d-maven

Maven distribution of the Slick2D Gava game development library
BSD 3-Clause "New" or "Revised" License
66 stars 36 forks source link

FIX avoid width and hight check for objectgroup in TiledMap #22

Closed HerrDerb closed 6 years ago

HerrDerb commented 6 years ago

The inner class ObjectGroup still checks for hight and width on an object layer and fails if not available. This attributes are no longer supported by Tiled as they make no sense.

Useage needs to be removed

Offering a PullRequest

nguillaumin commented 6 years ago

Hi, Slick hasn't really kept up with Tiled changes so it's indeed quite possible that there are inconsistencies like that.

Happy to take a pull request, but I guess it would need to cater for old Tiled format vs. new to be backwards compatible?

stefanhendriks commented 6 years ago

I believed Slick was dead and abandoned? If not, then it would be great to use this Repo to keep it up to date.

nguillaumin commented 6 years ago

Oh yeah it is abandoned. I setup this repo initially to publish Slick on Maven Central, but since then a few people have been submitting pull requests with bug fixes. Usually I merge them and publish a new version on Central.

HerrDerb commented 6 years ago

Fair enough. I've switched to https://github.com/mini2Dx

nguillaumin commented 6 years ago

No worries, looks like a cool library!

HerrDerb commented 6 years ago

Yes, it's quite a "slick" ;) library from a very active and responsive dev