Triplex

yes it fixes bg queue crashes when map updates are executed in threads. For single threaded core updates this patch doesn't change anything.

commited in [8799]. i tested it myself :mad:.

v0,6 - fixed compilation error. Thx for report and to vladimir and balrok for discussion .

added uint64 cast to prevent warning and possibly unexpected behaviour. that crash is about guild_bank, you can report bug. Edit: I tested patch, and fixed problems with leaving queues - i forgot "!" in one condition .... So patch is almost ready to be commited. New version added

updated to the latest rev. Pls test it, it would not be cool to apply patch without testing, as i do so often ....

yes client is able to display 255 or 254 (not sure..) mails, there is no way to allow more. On official when you have more than 255 mails in mailbox and you did read all first 255 you will still see envelope near map that you have new mail. You have to delete some mails to be able to see other. There is hidden timer on client, when he sends CSMG_MAIL_LIST or reads mails from latest cache - SMSG_MAIL_LIST_RESPONSE.

edited: Thx to Zor for help with mutex in BattleGroundMgr scheduler. Patch is ready to be tested. to Leak: This patch is based on Ambal's patch ... thx to him too...

hello, i created small change in bg queues code to prevent queue crashes with mtmaps patch. It is not tested, but it is able to build and it should work. version 0,6: http://paste2.org/p/504012 I 'm writing thread code for the first time, i know bases about how mutex / locks should work.... but i know nothing about ACE... Any suggestions are welcome.

commited in[8721]. Thx.

if i can request feature from you to add to your patch. It would be saving pet ids in battlegrounds - there are many conditions when pet is / isn't respawned after player is respawned by ressurection system in battleground. And patches that were here on mangos forum just added special variable to player class (last_pet_used_id) - but it is not correct, maybe we can use some other player's class variable - like lastsummonedpet or how it is called to respawn CORRECT pet after ressurection in bg.

I don't want to show off, but i think we created better bg queue system... :cool: There were many things that we decided that some people would not agree. For example new BG instance is created when there are at least minimum players in queue from each faction - some would object it is better to start when there are at least maximum players of both factions... There are 2 possible config options for bg queues behaviour in mangos.conf file. Default is much easier and allows bgs to be very unballanced, but the other value of option should keep bgs ballanced as much as possible. From my point of view bg queues don't require any big changes...

fixed in [8565], thx for report. That log errors are correct, and they doesn't cause any problems - players with bad zone data are added to guild as members, only error message is shown, you can disable that message by commenting it... ;-)

commited in [8560].

Implemented also arena_team loading speedup patch is available in: http://paste2.org/p/435312 patch is tested and it will be committed soon, any bug reports are welcome.

it is possible to play skirmish arena for 2 horde against 2 alliance. Your suggestion to allow 3 horde and 1 alliance would need to change code, you can do that for yourself, because it won't be commited to core.