PDA

Просмотр полной версии : [patch/dev] use DungeonEncounter.dbc


rsa
20.03.2011, 22:58
Если кому интересно, то я сегодня закончил портирование кода от Shauren, использующего DungeonEncounter.dbc. К сожалению, в чистом ядре кода, которому оно может понадобиться, нет вообще (у меня оно используется пока только для сообщения о биндинге групп, планируется в LFG), поэтому статус патча не определен. Да и процесс тестирования только пошел.
Патч размазан на несколько коммитов в моем форке, 2 основных:
загрузка данных об энкаунтерах:
https://github.com/rsa/mangos/commit/930504b802fd9ba353e39525a59aeeff41d140e7
save/load маски выполненных энкаунтеров, ее установка по спеллам или киллам (внимание! эта часть сделана совсем не как у Shauren и вообще не требует скриптов для работы)
https://github.com/rsa/mangos/commit/73abaf52a487056e7f1b064883fe44a7da162a49
ну и пример использования (с небольшим фиксом предыдущего коммита). very simple.
https://github.com/rsa/mangos/commit/ea8f938c4b173681fbef70a2342925163f5b552e
При наличии интереса team - сделаю для чистого ядра. Правда, поскольку интереса ни разу не было, то и делать пока не приходилось.

Laise
10.08.2011, 11:39
Просьба сделать для чистого ядра ^_^ и если не трудно запостить на гетмангосе тоже

rsa
10.08.2011, 12:21
Сорри, для чистого ядра сделать не выйдет, точнее выйдет - но только ту версию что в теме, а текущая уже здорово переработана. Примерно 3-4 предварительных имплементации придется сделать, строчек по 20 каждая. И их еще надо найти :) Работа (в любом варианте) довольно приличная, и без веских причин заниматься ей не очень охота - последние несколько "патчей на чистое", что я делал по просьбам мангос-девов, так и остались висеть в воздухе.

Laise
10.08.2011, 12:43
Сорри, для чистого ядра сделать не выйдет, точнее выйдет - но только ту версию что в теме, а текущая уже здорово переработана. Примерно 3-4 предварительных имплементации придется сделать, строчек по 20 каждая. И их еще надо найти :) Работа (в любом варианте) довольно приличная, и без веских причин заниматься ей не очень охота - последние несколько "патчей на чистое", что я делал по просьбам мангос-девов, так и остались висеть в воздухе.

если данная имплементация является не правильной - то тогда нет смысла что либо с ней делать. А какие именно нужны причины ? Сразу коммит - это вряд ли, это достаточно большое изменение чтобы дать какое то время на его тестирование.В общем вам решать, делать патч или нет, только скидывать на *разработчикам на все пофиг* потом не надо =)

rsa
10.08.2011, 13:13
если данная имплементация является не правильной - то тогда нет смысла что либо с ней делать. А какие именно нужны причины ? Сразу коммит - это вряд ли, это достаточно большое изменение чтобы дать какое то время на его тестирование.В общем вам решать, делать патч или нет, только скидывать на *разработчикам на все пофиг* потом не надо =)
она не неправильная, она неполная. в принципе конечно с нее можно начать, благо ее код в мангосе использовать сейчас просто совсем негде, а соответственно результаты тестирования будут блестящими :) не может плохо работать код, к которому нет обращений :)
"скидывать" ни на кого я ничего не собираюсь. весь мой код полностью открыт - заходите и берите. а вот прикладывать дополнительные усилия по прикручиванию - хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.
PS вот кстати еще один коммит из этой же серии:
https://github.com/rsa/mangos/commit/3a0bc5025d079adb67f1ccedc949d72a57ae4f71
уже не настолько нужен, но без него ЛФГ будет неполной. это уже чисто мое изобретение.

Laise
10.08.2011, 13:32
хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.

полную гарантию даст только страховой полис (с) Бендер.

---
https://github.com/rsa/mangos/commit...49d72a57ae4f71
может это лучше уже отдельным коммитом сразу перед самим лфг добавлять, хотя хз, надо ж хотя б основы закинуть

rsa
10.08.2011, 14:09
Браво :)
ну тогда еще несколько обязательных коммитиков в кучу (часть из них можно реализовать по другому но все равно придется). опять же, это оригинальные версии - текущие обычно переработаны.
https://github.com/rsa/mangos/commit/d2cde60d1a6f3e2bbdde31decc56c7d541173b82
https://github.com/rsa/mangos/commit/b71ffc6e4d332d86fc14ece413d40590c4ed07e7
https://github.com/rsa/mangos/commit/f676855ad0a8a6438bb95b80f31e5c9e32d8885d
https://github.com/rsa/mangos/commit/c56c3977f5b230ccdd0ec8a053ea057678527367 - этот частично уже в мэйнстриме
https://github.com/rsa/mangos/commit/dd25e3801db0960b4ae584fe047977f2855b0028

не смог найти - патч на обновление крич после смерти (сейчас новая версия от SilverIce)