* Improve CI; Add rustfmt & clippy jobs

* Run `cargo fmt --all`

* More permissive clippy job

* Apply clippy suggestions

* Run rustfmt again

* Add `Loader` and map loading functions

* Update examples, doc & tests

* Add tileset loading functions

* Fix unrelated broken intradoc link

* Fix intradoc link

* Fix more docs

* Update changelog and readme

* Fix warnings

* Change version, remove dead code

* Implement `Deref<Target = TData>` for `T`

* Move data properties to their respective data type

* Add docs to Data types

* Remove properties to user-side members

* Fix example/test builds

* Address PR comment

* Add map member properties

* Address PR comments

* Fix tests

* Address PR comment

* Add properties to `Object`

* Add properties to `FiniteTileLayer`

* Add properties to `Layer`

* Add properties to `ImageLayer`

* Add properties to `ObjectLayerData`

* Remove `data()`; Replace `map()`

* Add properties to `LayerTile`

* `MapWrapper` -> `map_wrapper!`

* Fix examples/tests

* Limit visibility of data types

* Make `Tile::collision` public again

* Doc tweak

Co-authored-by: Thorbjørn Lindeijer <bjorn@lindeijer.nl>

* Refactor chunk system
- Add `FiniteTileLayer` & `InfiniteTileLayer`
- Chunks are now stored by chunk position
- Add `InternalChunk` for holding mid-parse chunk info

* Fix tests

* Cleanup

* Make chunk constructor private

* Address PR comments

* Box chunk tiles

* Convert numeric types before multiplication

* Changed Rc to Arc

* Trying to trigger a rebuild

* Trying to trigger a rebuild

* Testing

* Add SFML example
Adds a rough demo using SFML for rendering.

* Simplifications to SFML example

* Rely on sfml 0.16 rather than a git revision
* Avoid allocating temporary Path
* Take into account margin and spacing
* Directly produce a FloatRect, since it is what SFML needs
* Renamed VertexMesh to QuadMesh
* Use Vertex::with_pos_coords since we always passed Color::WHITE
* Removed TileLayer wrapper, since it added no value
* Removed handling of gid == 0 in Tilesheet::tile_rect

* Update contributors list

* Apply formatting

* Avoid the need to inverse the camera matrix

* De-hardcode the tile size

Also fixes the example a bit, because the map was drawn too small.

* Update workflow

* Try fixing CI issue by removing rust-cache

* Fix CI

* Fix map asset path

* Merge SFML and Rust workflow

* Misc changes

* Merge upstream/master

* Update example to master

* Fix line endings

* Fixed comment

Co-authored-by: Thorbjørn Lindeijer <bjorn@lindeijer.nl>

* Adding group layer support

* Added test case and fixed parsing code

* Adding comma

* Adding missing newline

* Merging GID changes

* Merging GID changes

* Cleaning files

* Cleaning files

* Cleaning files

* Partial commit

* Partial commit

* Partial commit

* Some suggested changes and fixed tests (except infinite)

* Replaced `tile` member in `LayerTileRef` with a function to get an
  `Option<&Tile>`.

* Replaced `Map::get_tile_by_gid` with `Map::get_tileset_for_gid`, which
  just returns the `Option<&Tileset>`. It also does a reverse search,
  which fixes the lookup in case an external tileset has grown since the
  map was last saved.

* Replaced `Tileset::get_tile_by_gid` with `Tileset::get_tile`, since
  subtracting of the `first_gid` already happens when creating the
  `LayerTileRef`. Also, we eventually should remove `first_gid` from
  `Tileset`, since it should be possible to share a single tileset
  instance betweeen several maps.

* Pre-allocate the tiles hash map for the expected number of tiles and
  use `or_default` instead of `or_insert_with(Default::default)`.

* [nonbuilding] Move ownership of tilesets
- Moves the ownership of tilesets from Map to an object implementing `TilesetCache`

* Clean up

* More cleanup

* Organize layers into modules

* Further modularization

* Add layer wrappers

* Implement `Clone + PartialEq + Debug` for wrappers

* Fix example

* Fix all tests except for test_infinite_tileset

* Move layer utils to its own module

* Better `Map::layers` documentation

* `TilesetCache` -> `cache::ResourceCache`

* Add `ResourcePath`, rename and add errors

* Interface changes
- Move embedded tilesets from cache to map
- Store `Option<LayerTileData>` instead of `LayerTileData`

* parser ->`&mut impl Iterator<Item=XmlEventResult>`

* Document that tilesets are ordered by first GID

* Fix the layer tiles using GIDs issue

* Run `cargo fix`

* Fix `test_infinite_tileset` tests

* Implement a way to access object tile data

* Rename `TiledWrapper` to `MapWrapper`

* More efficient `get_or_try_insert_tileset_with`

* Add `ResourcePathBuf`, use `Rc<Tileset>` in `Map`

* Remove `MapTileset::first_gid`

* Run `cargo fix`

* Remove unrelated `Tileset` changes

* Requested changes

* Avoid reference counting when accessing tiles

* Store tile data instead of (u32, u32)` in objects

* Remove unneeded functions from `ResourceCache`

* Address PR comments

* Misc improvements in `layers::tile`

* Improve example

* Convert `LayerTile` properties to fields

Co-authored-by: Thorbjørn Lindeijer <bjorn@lindeijer.nl>

* Expose types at the top level

While the modules are great for organizing the code internally, it is more
convenient for users of the create when the types are available at the top
level. This avoids the verbosity like "map::Map".

* Changelog updated

Removed the lines stating where which class has been moved, since all these
classes are not available at the top level again.

Also removed some things which I didn't think we actually noteworthy for people
reading up on what had changed.

* Restored some changelog items

Removal of these items unrelated to how the data types are exposed.

Move parse functions to its respective types, rename `Colour` for consistency with PropertyValue::ColorValue, add documentation to map members, add Map::source, misc improvements

Something weird happens with slicing syntax. from_str changed to parse

Before this commit parse took an EventReader. Change so that it takes a
Buffer