Simplify 'create_default_tiles'

[richelbilderbeek]

Is your feature request related to a problem? Please describe.

Currently, create_default_tiles looks like this:

std::vector<tile> create_default_tiles() noexcept //!OCLINT indeed a function that is too long
{
  std::vector<tile> tiles;
  {tile t(2, 8, 0, 0, 0, tile_type::arctic, tile_id());tiles.push_back(t);}
  {tile t(4, 7, 0, 90, 0, tile_type::arctic, tile_id());tiles.push_back(t);}
  //...
  return tiles;
}

This can be simplified to:

std::vector<tile> create_default_tiles() noexcept //!OCLINT indeed a function that is too long
{
  return
  {
    tile t(2, 8, 0, 0, 0, tile_type::arctic),
    tile t(4, 7, 0, 90, 0, tile_type::arctic),
    //...
  };
}

Describe the solution you'd like

Do the simplification.

All tests must keep passing.

Describe alternatives you've considered

None.

Additional context

None.

[richelbilderbeek]

Well done :+1: