Code review comment for lp:~widelands-dev/widelands/seafaring-ai

Revision history for this message
GunChleoc (gunchleoc) wrote :

//NOCOM - how is it, when I added class here I have to cast everything

Because enough_ships isn't of the enum class' type. You will get rid of the cast if you do something like this:

enum class FleetStatus: uint8_t {kNeedShip = 0, kEnoughShips = 1, kDoNothing = 2 };

FleetStatus enough_ships = FleetStatus::kDoNothing;

etc...

Note that I also renamed fleetStatus to FleetStatus, because it's an Object class.

« Back to merge proposal