ApoC
Members-
Posts
87 -
Joined
-
Last visited
Never -
Donations
0.00 GBP
ApoC's Achievements
Advanced Member (3/3)
0
Reputation
-
I looked on to the patch (very quick look) and there are some problems i think. For example bool getUnloadLock() const { return i_unloadActiveLockCount || i_unloadExplicitLock; } ... void incUnloadActiveLock() { ++i_unloadActiveLockCount; } If we want to have this working under multithreaded environment it will need real locking (mutex or other primitive) because this and similar operations are not atomic what can cause problems in real multithreaded environment. Even simple ++ -- operations are not atomic! It is good to keep this in mind. It can be solved by some fast mutex or using atomic operations (special CPU instructions for this purposes).
-
[9651] [fix]improved quest POI's code + make use of actual data
ApoC replied to Auntie Mangos's topic in ... acceptedOld
Did you check if client crash on some quest? Nofantasy complains that this is reason why poiID is in DB.... -
Shouldn't be there some chance? May be better to implement it in *_loot_template way?
-
Why not implement it into Unit::SpellDamageBonus ?
-
Applied in [9505] Thank You
-
Applied in [9504], proc flags looks to be already in Thank You.
-
[update][9503] autot repeating spells behaviour
ApoC replied to Auntie Mangos's topic in ... acceptedOld
Applied in [9503] Thank You. -
[9651] [fix]improved quest POI's code + make use of actual data
ApoC replied to Auntie Mangos's topic in ... acceptedOld
Do we neeed to store in structure PoiId? It is alwayes counted from 0 to X by one. So i think index in vector == PoiId. -
In void Aura::HandleAuraDummy(bool apply, bool Real) you are not checking caster pointer to NULL what can cause possible crashes.
-
I'll recheck windows build tonight. I used official release 5.7.5 (not SVN checkout).
-
[Crash] [8661] Crash while a map is loading: Map Container
ApoC replied to a topic in OldBug reports
Please check if you don't have one object (guid) present in more than one pool. -
ace/config.h is generated from config.h.in by configure script. And config.h.in is generated by autotools, so you should try to do autoreconf --install --force again.
-
I created new patch file. Tested on FreeBSD 8.0-STABLE AMD64 and compilation + linking went well. Link: http://filebeam.com/aef8c096803b8dbf5363ce9ff46ebfbc
-
Patch not working: 1. Missing some files from ACE distribution 2. Messed up Makefile.am 3. Why are everywhere removed version ids from all ace files? 4. Not compile on FreeBSD because of missing hack added by Derex I am now trying to fix it up
-
Why not loop in Spell::CheckCast aura effects like: Unit* target = m_targets.GetUnitTarget(); for (int i = 0; target && i < 3; ++i) { if (!m_spellInfo->EffectApplyAuraName[i]) continue; // else const AuraList& aurasByName = target->GetAurasByType(m_spellInfo->EffectApplyAuraName[i]); AuraList::const_iterator it = aurasByName.begin(); for ( ; it != aurasByName.end(); ++it) { // Do your checks here // on first hit return SPELL_FAILED_AURA_BOUNCED } } And may be better to do this as method for avoid code duplication on more places.
Contact Us
To contact us
click here
You can also email us at [email protected]
Privacy Policy | Terms & Conditions
You can also email us at [email protected]
Privacy Policy | Terms & Conditions
Copyright © getMaNGOS. All rights Reserved.
This website is in no way associated with or endorsed by Blizzard Entertainment®
This website is in no way associated with or endorsed by Blizzard Entertainment®