S3_0 Qt-client city_dialog::next_city() and city_dialog::prev_city() may even crash if the player has no cities (presumably one should have no city dialog to click next/prev from when that's the case, but with unlucky timing it's possible)
- City list size is against exactly 1 (meaning that the open city dialog is about the only city), not catching the empty city list case
- The next city is assigned in a meaningless loop, that just overwrites previous round result until it reach the city list size
S3_0 Qt-client city_dialog::next_city() and city_dialog::prev_city() may even crash if the player has no cities (presumably one should have no city dialog to click next/prev from when that's the case, but with unlucky timing it's possible)
- City list size is against exactly 1 (meaning that the open city dialog is about the only city), not catching the empty city list case
- The next city is assigned in a meaningless loop, that just overwrites previous round result until it reach the city list size