Random is good, at least we have some variety
Hmm, looking at this, the problem is that on receipt of CMSG_LOOT_ROLL we've lost direct visibility of what the roll is for (and it seems difficult to get that info from what's in that message) - it may not be a complete disaster, but not clear how to find it without keeping the PlayerBot code nicely encapsulated...unless we do something with the outbound message to the client - SMSG_LOOT_START_ROLL has all the details we need - including the itemid from which can be derived the ItemPrototype...
If someone can tell me I'm wrong re CMSG_LOOT_ROLL params (as I am new to the vast mangos codebase) then great, otherwise, I'll figure something else out...
Right now I'm focussing on being able to understand the info in ItemPrototype and how it can be used (ItemHandler/ObjectMgr/Player all seem to have examples), then I can figure out...do we want to wait for the "Master" to respond and then have the bots determine what they should do...this would enable us to say, for example, if the Master "Needs" then the bots pass...If we want to do that, it involves storing the item info between the outbound packet and the associated response (if one comes back...) and of course managing and cleaning up that data...