Data type inconsistency for tracker tiers

[thalieht]

libtorrent version (or branch):
Current master (8e53eba)

libtorrent/include/libtorrent/aux_/announce_entry.hpp

Line 181 in 8e53eba

 std::uint8_t tier = 0; 

libtorrent/include/libtorrent/add_torrent_params.hpp

Line 134 in 8e53eba

 std::vector<int> tracker_tiers; 

[Arvid Norberg]

do you think it's worthwhile to support more than 255 tiers?

[Vladimir Golovnev]

do you think it's worthwhile to support more than 255 tiers?

Personally, I don't like such artificial restrictions. Given the fact that it might be convenient for some user to assign a separate tier to each tracker, then I would keep their maximum number the same.