VladimirMangos

quest add crahs must ne fixed in [11448]

Rewrited version added in [11446]. Thank you

it can't be related guid: you will can't load character if guid will wrong. And "something" not very good description .... Maybe problematic characters have data in table 'character_battleground_data'. Can you check this for next problematic case?

In fact problem mostly related to quest start item. Must be fixed in [11438]. Thank you for report.

In [11436]. Thank you

"#3 0x00000000007d062f in Unit::SendThreatRemove (this=0x7fff0a269970," long ago reported crash. As i remember none one answer in original thread about what must happens with threat (not remember direct fix problem) but remember that from this has been dependent how need fix it. None answer, none time implement wrong way hacks

Also SD2 CMakeLists.txt not expected use for independent use with cmake. add dependence as described in SD2 docs, and then use cmake (not at windows case) with mangos root dir CMakeLists.txt

But some from its need all targets like http://www.wowhead.com/spell=52364 and some other as i understand. we can ofc made umax set to 1 with exception list. Note: some spells have in dbc exlicit 1 max target setting already but too small amount for been useful.

Hmm, as i look most reports about destructor calling for global singletons. valgrind can incorrect false detect problems maybe... Because related fields cleanup in code in singleton destructors... BattleGroundMgr::DeleteClientVisibleInstanceId case uninitilized conditinal case strange but i will change cdeo: it not need check in this point.

Possible, we have 2 diff cases: 1) client side send texts, including stored in DB. It's dependent from client locale (that can be != server locale). And gettext low chance can be used in this case. mangos_strings + localization shadow table in DB is good existed sollution. 2) server side only texts (console/etc). Adding tis all to mangos_strings can be too havy task, maybe in this case better use as in other similar cases gettext way or alt. tool if it portable work for suported by mangos platforms, specially win/some wide linux/some wide unixs

mangos have builting way add localization for texts. Maybe not so good automated as some existed tools but from other side that can be used for case: mangos_string table. Most texts send to client already support localizations. Internal error reporting to console logs mostly not localized just because devs and expected most admins know english in some acceptable level for understand this error messages.

In fact possible in legal free way get x64 support in small hacky way. MS Platform SDK include compiler that support x64 and possible integrate it with VS IDE. I think this has been described in some from thread and possible easy find by google also.

cobopoints removed at success spell cast finish: Spell::finish (also used in Spell::CheckCast) This has been discussed sometime ago and planned way implement shas been suggested: remove and use cobopoints for m_caster->GetCharmerOrOwnerPlayerOrPlayerItself() because non-players not have combopoints. But not has been realy implemented because lack possibility test without vehicles in normal way.

Instead modify code, maybe Db miss for creature UNIT_FLAG_PASSIVE = 0x00000200, // makes you unable to attack everything. Almost identical to our "civilian"-term. Will ignore it's surroundings and not engage in combat unless "called upon" or engaged by another unit. in flags field.

In fact _DB_ must have proper class settings for pet in creature_template. Current setting in code just hacks for prevent use wrong data until DB will have most cases propertly set and instead this hacks will possible add check at table loading with error reporting. In [11410] in master added alt. fix. I will donwload soon to One also.

In [11409]. Thank you

"c++ 2010 express edition" not have x64 support by default in package.

Crash if run not in wow dir. But more bad news. For me fail patch 4.1.0 exe (( Addition info: C# version also fail patch. So tool required offset or more deep updates (

Most git_id code for automate sql updates registrations in commit chnages and guards adding.

Nice, as i say better have similar programming language use for linked projects/tools for decrease problems for users. License as i understand must be preserved. But if none objections core replaced by C++. Then we not need binaries maybe and less legal points then.

This only meaning that we seelct way how will around client connection problem.

and ad.exe mostly not related to any quests work. At least directly. Start search problem from ad.exe hmm... look strange.

No universal tool ad.exe exist in world. _Different_ versions of ad.exe for _each_ client version exists. And its will not work with diff client verison. Need use exactly ad.exe from used repo or build own version for provided in used repo sources.

nice. Next question: currently it C# program, we can ofc use it as-is but if it will not include binary version then will create problems for ppl - req. additional VS component installed. So is it ok have binary form with project? if ok, then we can possible just use as-is, right?

Used wrapper hide differences in this part for diff client versions dbc. In past has been special field that used instead 1. In 4.x as i remember no -1 and dbc recalculated use real value. If use CalculateSimpleValue() then we can update only this function and get for same code same results for diff client versions in expected for client formula.