Ru-MaNGOS

Ru-MaNGOS (http://mangos.ytdb.ru/index.php)
-   Патчи (http://mangos.ytdb.ru/forumdisplay.php?f=6)
-   -   [patch/dev] use DungeonEncounter.dbc (http://mangos.ytdb.ru/showthread.php?t=3939)

rsa 20.03.2011 22:58

[patch/dev] use DungeonEncounter.dbc
 
Если кому интересно, то я сегодня закончил портирование кода от Shauren, использующего DungeonEncounter.dbc. К сожалению, в чистом ядре кода, которому оно может понадобиться, нет вообще (у меня оно используется пока только для сообщения о биндинге групп, планируется в LFG), поэтому статус патча не определен. Да и процесс тестирования только пошел.
Патч размазан на несколько коммитов в моем форке, 2 основных:
загрузка данных об энкаунтерах:
https://github.com/rsa/mangos/commit...9aeeff41d140e7
save/load маски выполненных энкаунтеров, ее установка по спеллам или киллам (внимание! эта часть сделана совсем не как у Shauren и вообще не требует скриптов для работы)
https://github.com/rsa/mangos/commit...fe44a7da162a49
ну и пример использования (с небольшим фиксом предыдущего коммита). very simple.
https://github.com/rsa/mangos/commit...2925163f5b552e
При наличии интереса team - сделаю для чистого ядра. Правда, поскольку интереса ни разу не было, то и делать пока не приходилось.

Laise 10.08.2011 11:39

Просьба сделать для чистого ядра ^_^ и если не трудно запостить на гетмангосе тоже

rsa 10.08.2011 12:21

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

Laise 10.08.2011 12:43

Цитата:

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

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

rsa 10.08.2011 13:13

Цитата:

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

она не неправильная, она неполная. в принципе конечно с нее можно начать, благо ее код в мангосе использовать сейчас просто совсем негде, а соответственно результаты тестирования будут блестящими :) не может плохо работать код, к которому нет обращений :)
"скидывать" ни на кого я ничего не собираюсь. весь мой код полностью открыт - заходите и берите. а вот прикладывать дополнительные усилия по прикручиванию - хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.
PS вот кстати еще один коммит из этой же серии:
https://github.com/rsa/mangos/commit...49d72a57ae4f71
уже не настолько нужен, но без него ЛФГ будет неполной. это уже чисто мое изобретение.

Laise 10.08.2011 13:32

Цитата:

Сообщение от rsa;
хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.

Цитата:

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

---
может это лучше уже отдельным коммитом сразу перед самим лфг добавлять, хотя хз, надо ж хотя б основы закинуть

rsa 10.08.2011 14:09

Браво :)
ну тогда еще несколько обязательных коммитиков в кучу (часть из них можно реализовать по другому но все равно придется). опять же, это оригинальные версии - текущие обычно переработаны.
https://github.com/rsa/mangos/commit...56c7d541173b82
https://github.com/rsa/mangos/commit...d40590c4ed07e7
https://github.com/rsa/mangos/commit...1e5c9e32d8885d
https://github.com/rsa/mangos/commit...ea057678527367 - этот частично уже в мэйнстриме
https://github.com/rsa/mangos/commit...7977f2855b0028

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


Текущее время: 16:49. Часовой пояс GMT +3.

ru-mangos.ru - Русское сообщество MaNGOS