[10935] Move script calls to ScriptMgr

[newsbot]

автор: zergtmn

m configure.ac
m src/bindings/Makefile.am
m src/game/BattleGroundHandler.cpp
m src/game/ChatHandler.cpp
m src/game/CreatureAISelector.cpp
m src/game/GameObject.cpp
m src/game/Level3.cpp
m src/game/Makefile.am
m src/game/Map.cpp
m src/game/MiscHandler.cpp
m src/game/NPCHandler.cpp
m src/game/QuestHandler.cpp
- src/game/ScriptCalls.cpp
- src/game/ScriptCalls.h
m src/game/ScriptMgr.cpp
m src/game/ScriptMgr.h
m src/game/SpellAuras.cpp
m src/game/SpellEffects.cpp
m src/game/SpellHandler.cpp
m src/game/Transports.cpp
m src/game/UnitAuraProcHandler.cpp
m src/game/WaypointMovementGenerator.cpp
m src/game/World.cpp
m src/mangosd/CliRunnable.cpp
m src/mangosd/Makefile.am
m src/mangosd/Master.cpp
m src/shared/revision_nr.h
m win/VC100/game.vcxproj
m win/VC100/game.vcxproj.filters
m win/mangosdVC100.sln
m win/mangosdVC80.sln
m win/mangosdVC90.sln

[10935] Move script calls to ScriptMgr



- Script library presence is now optional.

- Some script hooks have new names. Scripting libraries need to be adjusted accordingly.



Signed-off-by: zergtmn 

Дальше...

[virusav]

При компиляции вылезли ошибки из-за:

			<File
				RelativePath="..\..\src\game\ScriptCalls.cpp"
				>
			</File>
			<File
				RelativePath="..\..\src\game\ScriptCalls.h"
				>
			</File>

в game.vcproj.

Компилю VC90, в VC80 должна быть та же потеха.

Было бы неплохо править проекты не только VC100, если, конечно, косяк не мой.

[rsa]

Фигня это все... Из-за того что многие имена вызовов поменялись (часть имен вообще удалена, изменен формат части вызовов), а никакой отладочной информации на предмет их совпадений с имеющимися скриптами в лог не предусмотрено, сейчас многие скрипты просто перестанут работать. Безо всяких объяснений. Придется проверять все и вручную, это как минимум те скрипты где висят хуки на QuestReward и GOHello. Сотни 2-3. Мелочи...
4й раз за этот год меняется структура IPC. В хороших проектах она не меняется десятилетиями...

[virusav]

Если структура была неправильной, то менять ее все равно надо, даже с такими последствиями.
Думаю, что не проблема поправить, есть знаешь, что делаешь.

[zergtmn]

Переименованы хуки, названия для которых были выбраны неудачно и/или не соответствовали их назначению. QuestComplete дублировал QuestRewarded, поэтому был удален. ItemHello вообще никто не знал зачем нужен...
Чтобы пофиксить ScriptDev2 у меня ушло максимум минут 10.

Сотни 2-3

Зачем? Досточно переименовать функции, которые напрямую вызываются из DLL.

[rsa]

Сообщение от virusav

Если структура была неправильной, то менять ее все равно надо, даже с такими последствиями.
Думаю, что не проблема поправить, есть знаешь, что делаешь.

чтобы менять структуру, надо сначала родить Calling Convention. без нее как был бардак, так и останется. завтра придет кто-то еще и ему (как мне) не понравится разделение вызовов GOGossip/Gossip, опять будем бегать с намыленными задницами.

Добавлено через 7 минут

Сообщение от zergtmn

Переименованы хуки, названия для которых были выбраны неудачно и/или не соответствовали их назначению. QuestComplete дублировал QuestRewarded, поэтому был удален. ItemHello вообще никто не знал зачем нужен...
Чтобы пофиксить ScriptDev2 у меня ушло максимум минут 10.

Зачем? Досточно переименовать функции, которые напрямую вызываются из DLL.

с первой частью все просто, хотя зачем было убивать slot непонятно. не мешал ведь никому.
а вот насчет переименовать - это ничего, что GOHello, GOGossipHello и GOUse это _три_ (а не две) разные вещи, по крайней мере у меня? и это не моя идея, а раньше было захардкодено... короче после тупого переименования примерно 70 скриптов улетело. после хакания scriptmgr.cpp половина вернулась. осталось разбираться почему не пашет около 20 скриптов на QuestRewarded, для них вроде все должно было пройти корректно, но не пашут и все...

[zergtmn]

GOHello это старое незвание GOUse. Параметры не изменились. GOGossipHello не менялся.
Вот патч для sd2 http://paste2.org/p/1169267, все сигнатуры соответствуют используемым в ядре.
QuestRewarded я проверял на квесте 7603.

[virusav]

rsa, не отвлекай zergtmn, пусть транспорт реализует.

[rsa]

Сообщение от zergtmn

GOHello это старое незвание GOUse. Параметры не изменились. GOGossipHello не менялся.

вот именно из-за этого заблуждения у меня много чего и не работало. это не GOUse, а дубль к GOGossipHello.
Тезка, ты хоть проверяй патчи-то... После твоего патча например пилоны в Унгоро сдохнут насовсем.

Добавлено через 1 минуту

Сообщение от virusav

rsa, не отвлекай zergtmn, пусть транспорт реализует.

хе. у меня его транспорт уже 4 месяца как шуршит вовсю, уже скриптами оброс как пароход после кругосветки...

[zergtmn]

Сообщение от rsa

это не GOUse, а дубль к GOGossipHello.

Внимательнее смотрим код: http://paste2.org/p/1169333
OnGameObjectUse вызывает GOUse(Player*, GameObject*) из DLL.

После твоего патча например пилоны в Унгоро сдохнут насовсем.

Специально проверил, квесты работают как и раньше.
В этом коммите в основном рефакторинг, сломаться ничего не должно. К тому же, я выборочно тестировал скрипты из sd2 перед коммитом.

[Ambal]

ИМХО, такие изменения под большие праздники лучше не коммитить, т.к. вероятность что-то поломать есть всегда, а возможность оперативно исправить стремится к нулю.

P.S. Предлагаю не устраивать здесь Ледовое Побоище накануне Нового Года. Что сделано, то сделано - если вы не предлагаете решения возникших проблем, то убедительная просьба не троллить на форуме и не портить всем настроение.

[zergtmn]

Где проблемы то? Вижу только недопонимание у rsa.