Облик стремительной птицы

[xmolex]

Я уже давно дружу с Mangos, но до сих пор никак не могу понять одной вещи, поэтому решил всетаки спросить:
Вот есть такой спелл: Облик стремительной птицы (http://ru.wowhead.com/?spell=40120), скорость в этой форме у персонажа не меняется. Эта ошибка уже наверное пол года, хотя я не могу сказать, что это неважный баг. Но суть не в этом. На getmangos.com есть патч, повышающий скорость до нужной.

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 18b4af6..e752927 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -256,7 +256,7 @@ pAuraHandler AuraHandler[TOTAL_AURAS]=
     &Aura::HandleNoImmediateEffect,                         //203 SPELL_AURA_MOD_ATTACKER_MELEE_CRIT_DAMAGE  implemented in Unit::CalculateMeleeDamage and Unit::SpellC
     &Aura::HandleNoImmediateEffect,                         //204 SPELL_AURA_MOD_ATTACKER_RANGED_CRIT_DAMAGE implemented in Unit::CalculateMeleeDamage and Unit::SpellC
     &Aura::HandleNoImmediateEffect,                         //205 SPELL_AURA_MOD_ATTACKER_SPELL_CRIT_DAMAGE  implemented in Unit::SpellCriticalDamageBonus
-    &Aura::HandleNULL,                                      //206 SPELL_AURA_MOD_SPEED_MOUNTED
+    &Aura::HandleAuraModIncreaseFlightSpeed,                //206 SPELL_AURA_MOD_SPEED_MOUNTED
     &Aura::HandleAuraModIncreaseFlightSpeed,                //207 SPELL_AURA_MOD_INCREASE_FLIGHT_SPEED
     &Aura::HandleAuraModIncreaseFlightSpeed,                //208 SPELL_AURA_MOD_SPEED_FLIGHT, used only in spell: Flight Form (Passive)
     &Aura::HandleAuraModIncreaseFlightSpeed,                //209 SPELL_AURA_MOD_FLIGHT_SPEED_ALWAYS
diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index fbcf302..83bef21 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -10784,7 +10784,7 @@ void Unit::UpdateSpeed(UnitMoveType mtype, bool forced)
             if (IsMounted()) // Use on mount auras
                 main_speed_mod  = GetMaxPositiveAuraModifier(SPELL_AURA_MOD_INCREASE_FLIGHT_SPEED);
             else             // Use not mount (shapeshift for example) auras (should stack)
-                main_speed_mod  = GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_FLIGHT);
+                main_speed_mod  = GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_FLIGHT) + GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_MOUNTED);
             stack_bonus     = GetTotalAuraMultiplier(SPELL_AURA_MOD_FLIGHT_SPEED_ALWAYS);
             non_stack_bonus = (100.0f + GetMaxPositiveAuraModifier(SPELL_AURA_MOD_FLIGHT_SPEED_NOT_STACK))/100.0f;
             break;

Так вот сам вопрос. Патч ведь рабочий и вроде по коду нормальный. Но так как его не добавляют, то разумно предположить, что это хак. Либо офф команде не хватает людей, чтобы его заметить. Хотя за такое количество времени думаю его видели, поэтому скорее всего верно первое.
1) Если это хак, то как вы поняли, что это хак?
2) Кто, вообще, решает быть этому или не быть частью ядра?
3) Если допустим Vladimir добавит некий код в исходники, другие разработчики из сообщества наврятли будут его проверять. Или будут?
4) Тоесть, если ты хочешь, чтобы твой патч попал в проект, нужно не только его грамотно написать, но и указать на его кому-нибудь из сообщества? Или просто разработчиков, которые смотрят патчи других людей не так много, а вернее один Vladimir?

Кстати замечу что с патча 3.2, облик стремительной птицы может давать скорость 310% при наличии у вас хотя одного маунта 310%

[xmolex]

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

Кстати замечу что с патча 3.2, облик стремительной птицы может давать скорость 310% при наличии у вас хотя одного маунта 310%

Тоесть, поэтому это хак?

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

Тоесть, поэтому это хак?

Нет, я просто уточнил механику спела

[Fedia22]

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

[xmolex]

Понимаете, я говорю сейчас не про русскоязычный форум, я говорю про весь проект. Я хотел бы внести свою лепту в развитие проекта, но мне непонятен отбор. Что-то не так меня не устраивает, я хочу писать патчи, которые будут приняты. Вышеуказанный патч написан не мной и висит уже полгода на getmangos.com. Все его используют, но добавить его не добавляют. Поэтому я и хотел выяснить, почему? Что не так?

[YuruY]

http://ru-mangos.ru/showthread.php?t=31

[xmolex]

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

http://ru-mangos.ru/showthread.php?t=31

"Тема не существует или не указан идентификатор (номер)."

[YuruY]

У меня открывает:

Ru-MaNGOS > Ядро > Патчи > Патчи на рассмотрении
Тема: "Шанс по рассмотрению патчей" (на 2 странице раздеа)

[xmolex]

Наверное у меня прав недостаточно или нужны какие-то хитрые настройки в профиле, но темы нет. Даже поиск ее не выдает.
Под гостем темы также нет, значит наверное прав всетаки нет.

[tempura]

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

Наверное у меня прав недостаточно или нужны какие-то хитрые настройки в профиле, но темы нет. Даже поиск ее не выдает.
Под гостем темы также нет, значит наверное прав всетаки нет.

исправлено. читайте пожалуйста.

[xmolex]

Я прочитал.
Дело в том, что вышеизложенный патч и так висит пол года в under review на getmangos.com. Как раз поэтому я и хочу поинтересоваться, что это - нехватка рук на проверку или кривой патч? Я бы задал вопрос там, но не очень изъясняюсь на английском, поэтому решил здесь.
Просто что толку писать патчи, если их всеравно никто рассматривать не будет или писать так, что ты будешь думать, что все вроде нормально, а на самом деле что-то не так. Вот я взял рабочий, вроде не хаковый патч, который висит уже пол года и эти пол года он нормально выполняет свою функцию и хочу ответов, что не так?

[DiffuSer]

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

[xmolex]

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

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

Это все обстрактные точки зрения, которые и так понятны. Я привел конкретный патч, в котором кода с гулькин нос, специально, чтобы разобраться. Просто если патч нормальный, но на него тупо забили, то это наводит на некоторые мысли.

[Fedia22]

перечитайте ещё раз пост DiffuSer'а... и читайте до тех пор пока не поймёте

[xmolex]

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

перечитайте ещё раз пост DiffuSer'а... и читайте до тех пор пока не поймёте

Тоесть конкретно в этом патче нет комментариев, и поэтому его пол года шлют лесом?
Или вы предлагаете приглашать к себе Vladimir'а, угощать его пивом и в процессе поглощения сего напитка, объяснять каждую строку своего патча? :-)

Меня не устраивает ответ: "наверное с ним что-то не так". Я хочу знать, ЧТО не так конкретно с этим патчем! Я выбрал его не случайно. Я считаю, что он вполне нормален и достоин того, чтобы быть добавленным в проект. Но раз его не добавляют, значит есть причина. И эта причина - священная тайна. Извините, но если это священная тайна, то как другим писать патчи, если их будут просто посылать лесом без аргументов?

Я заинтересован в проекте не меньше официального сообщества, но как играть по правилам, если эти правила от тебя скрывают?

[Vladimir]

в getmagos.com under review ~400 патчей, и если вы заметили по логам github.com последние дни я не работал практически над master-веткой, а занимался переносом коммитов из mangos0.12 -> mangoszero. Для увереного ответа правильный или нет этот патч надо смотреть другие спелы с этой аурой и т.д. Я этого не делал.

И если этот патч уже прредложен в getmangos.com, то п-о-ч-е-м-у не приведен на нее хотя-бы линк... Я как должен определять где более свежая версия или кто автор, особенно если ники разные и т.д....

[xmolex]

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

п-о-ч-е-м-у не приведен на нее хотя-бы линк...

Извините.
http://getmangos.com/community/showt...id-Flight-Form
Я никого не обвиняю, я просто хочу ответов.

[tempura]

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

Извините.
http://getmangos.com/community/showt...id-Flight-Form
Я никого не обвиняю, я просто хочу ответов.

Не извиняю. Вы именно всех вокруг обвиняете, и полностью отказываетесь от каких-либо ответов.
Если бы вы хотели получить ответ - вы должны были верно задать вопрос. На ваш вопрос "вот какой-то патч, какого хрена он не в репо?" - последовали вполне закономерные ответы. Если бы вы изволили вместо этого заняться поиском ВСЕХ подобных патчей, реализующих данную возможность, выложили бы ссылки на них и к каждому приложили свое описание как именно они работают, в чем их положительные стороны, и в чем отрицательные - не сомневайтесь, с самого начала ветка выглядела бы совершенно иначе.

[xmolex]

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

Не извиняю.

Дело не совсем так. Мне на этот патч параллельно. Просто это частный случай. Я посчитал, что лучше привести что-то, чем вообще ничего.
Я бы хотел отдать часть своего времени на помощь проекту в виде написания и публикации функционала в виде патчей, т.к. писать и постоянно подгонять их под новые ревизии просто надоело. Хочу начать публиковаться. Но хочу в начале выяснить, а стоит ли? Может мои патчи будут также лежать где-нибудь и пылиться, хоть они и будут нормальными и вся моя помощь переедет в подгонку их под новые ревизии в надежде, что когда-нибудь их заметят. Жизнь одна и тратить ее нужно мудро.

[YuruY]

xmolex
Я кстате на это намекнул вам в посте #7, следовало просто прочитать пост по данной мною ссылке (именно 2 абзац).

[tempura]

Это как раз конкретный случай. Для общих случаев уже есть описание и объяснение: http://ru-mangos.ru/showthread.php?t=31

А насчет вопроса "будут ли мои патчи приняты в репо"... Знаешь, твой подход "если примут - то буду писать, а если не примут - не буду" выглядит лично для меня как-то странно. Давай лучше так - ты полностью с нуля все сам напишешь, у тебя все будет работать лучше всех, и ты сам будешь решать что тебе куда принимать, а что нет?
Я не разбираюсь в проекте, но я вижу на гетмангос-ком количество принятых-отклоненных патчей, и патчей на рассмотрении. Какую-то экстраполяцию я могу сделать. Если бы разбирался - открыл бы в отвергнутых патчах тему у которой внизу есть ответ "принят альтернативный патч", взял бы их два и сравнил - почему один принят, а второй нет.

[xmolex]

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

...

Если нам нужно получить результат сложения 1 и 2 и есть патч с кодом a=1+2; ,который будет лежать пол года на getmangos.org и на вопросы почему его не применяют все будут говорить, "выложили бы ссылки на них (другие патчи реализующие такой же функционал) и к каждому приложили свое описание как именно они работают, в чем их положительные стороны, и в чем отрицательные", то я не хочу в этом участвовать.

Я специально привел патч, который:
1) Лежит уже долгое время без принятия
2) Который минимум в коде и все в нем интуитивно понятно даже не программисту
3) Который используется почти во всех сборках и реализует то, что должен уже очень продолжительное время

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

Мне обидно за скорость развития проекта. Ее можно было бы очень повысить, если выработать правила игры.

[virusav]

Не все так плохо, как кажется.

Я, например, решил писать скрипты на СД2, заодно начать изучения Си++.
В итоге первые 2 патча были признаны разработчиками СД2 рабочими, но не были приняты по их собственным причинам.
Но меня это не остановило, продолжаю дальше работу и выкладываю скрипты им, использую их сам.

Если хотите внести свой вклад в проект, вносите, пишите патчи, скрипты и т.д.

[tempura]

Мне почему-то кажется, что добиваться одного эффекта сложением двух аур - это как-то не совсем прилично. Вероятно должен быть какой-то иной путь для реализации. Как я понимаю - основной идеей проекта является не "воткнем что попало как-нибудь и пусть работает". Тут наверное реализация какая-то требуется.

А пример со сложением вообще некорректен в данном случае, так как здесь нам надо складывать не банальные единицы, а килограмм тротила, метр хлопка и час забора чтобы получить литр пива.

[xmolex]

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

Мне почему-то кажется, что добиваться одного эффекта сложением двух аур - это как-то не совсем прилично. Вероятно должен быть какой-то иной путь для реализации. Как я понимаю - основной идеей проекта является не "воткнем что попало как-нибудь и пусть работает". Тут наверное реализация какая-то требуется.

Вот это другой разговор, спасибо.

[xmolex]

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

...

Ваше пожелание учтено. По вашим словам если патчи будут вида, как ниже, то их не будут динамить. Я вас правильно понял?

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 3ee1892..87e3787 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -256,7 +256,7 @@ pAuraHandler AuraHandler[TOTAL_AURAS]=
     &Aura::HandleNoImmediateEffect,                         //203 SPELL_AURA_MOD_ATTACKER_MELEE_CRIT_DAMAGE  implemented in Unit::CalculateMeleeDamage and Unit::SpellC
     &Aura::HandleNoImmediateEffect,                         //204 SPELL_AURA_MOD_ATTACKER_RANGED_CRIT_DAMAGE implemented in Unit::CalculateMeleeDamage and Unit::SpellC
     &Aura::HandleNoImmediateEffect,                         //205 SPELL_AURA_MOD_ATTACKER_SPELL_CRIT_DAMAGE  implemented in Unit::SpellCriticalDamageBonus
-    &Aura::HandleNULL,                                      //206 SPELL_AURA_MOD_SPEED_MOUNTED
+    &Aura::HandleAuraModIncreaseFlightSpeed,                //206 SPELL_AURA_MOD_SPEED_MOUNTED
     &Aura::HandleAuraModIncreaseFlightSpeed,                //207 SPELL_AURA_MOD_INCREASE_FLIGHT_SPEED
     &Aura::HandleAuraModIncreaseFlightSpeed,                //208 SPELL_AURA_MOD_SPEED_FLIGHT, used only in spell: Flight Form (Passive)
     &Aura::HandleAuraModIncreaseFlightSpeed,                //209 SPELL_AURA_MOD_FLIGHT_SPEED_ALWAYS
diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index b9af5d0..451b6a2 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -10781,7 +10781,12 @@ void Unit::UpdateSpeed(UnitMoveType mtype, bool forced, float ratio)
             if (IsMounted()) // Use on mount auras
                 main_speed_mod  = GetMaxPositiveAuraModifier(SPELL_AURA_MOD_INCREASE_FLIGHT_SPEED);
             else             // Use not mount (shapeshift for example) auras (should stack)
-                main_speed_mod  = GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_FLIGHT);
+            {
+               if (Unit::HasAuraType(SPELL_AURA_MOD_SPEED_MOUNTED)) // shapeshift
+                   main_speed_mod  = GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_MOUNTED);
+               else
+                   main_speed_mod  = GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_FLIGHT);
+            }
             stack_bonus     = GetTotalAuraMultiplier(SPELL_AURA_MOD_FLIGHT_SPEED_ALWAYS);
             non_stack_bonus = (100.0f + GetMaxPositiveAuraModifier(SPELL_AURA_MOD_FLIGHT_SPEED_NOT_STACK))/100.0f;
             break;

В SpellAuras.cpp указываем что ауру SPELL_AURA_MOD_SPEED_MOUNTED нужно обрабатывать в Aura::HandleAuraModIncreaseFlightSpeed

В Unit.cpp определяемся со скоростью. Из Aura::HandleAuraModIncreaseFlightSpeed мы получили mtype MOVE_FLIGHT для функции Unit::UpdateSpeed. Соответственно, если этот mtype, то выполняем блок. В блоке два условия, если мы примонтированы и нет. При shapeshift мы не монтируемся, поэтому попадаем в блок else. Ну и теперь мы смотрим, если у нас есть аура SPELL_AURA_MOD_SPEED_MOUNTED (if (Unit::HasAuraType(SPELL_AURA_MOD_SPEED_MOUNTED))) , то мы получаем для нее скорость. Если нет, то работаем как и работали ранее.

Патч работает корректно, протестировал Облик стремительной птицы (http://ru.wowhead.com/?spell=40120) и Облик птицы (http://ru.wowhead.com/?spell=33943). Скорость высталяется пропорционально скиллам.

[tempura]

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

Ваше пожелание учтено. По вашим словам если патчи будут вида, как ниже, то их не будут динамить. Я вас правильно понял?

Первое, что следовало бы понять по моим словам - что я не разбираюсь в программировании в общем, и в проекте мангос в частности. Я специально делал упор на этом.
Патчи рассматриваю совсем не я, так что считать "если темпура сказала, то все побежали делать" - глубочайшее заблуждение.

Однако в той области, где я работаю, принято представлять свои варианты именно в таком видет. Чтобы человек не просто увидел текст и сказав "шоза фигня" закрыл и забыл, а получил удовольствие от того, что для него, для его понимания, все расписано досконально.

Когда ко мне приходят со словами "ничего не трогал - само сломалось" - мне просто впадлу что-то делать. А когда сообщают "сделал то-то, получилось то-то, и тут все как грохнуло!" - тогда я делаю все с удовольствием.

[ANTOMA]

Ребята, хватит гнобить парня.... Задан прямой вопрос, что нужно для...
Владими написал что требуется, по остальному не стисняйся, пиши по аналогии с подобными вещами уже реализоваными в коде, при этом проверяй не только спел который исправляешь, но и возможные проблемы, который данный патч (потенциально) может вызвать...
Лучший способ, проверка на лайф сервере в пару дней.. и не парься, и не оправдывайся, все у тебя получится.

[xmolex]

Хотелось бы чтобы всетаки Vladimir хоть намекнул, что же всетаки не то в патче, т.к. вторая версия - это просто более красивая первая, а суть то одна.

[Warlord123]

Понятно, что Vladimir'у просмотреть и проверить более 100 патчей на ru-mangos.ru и более 400 на getmangos.com нет возможности. По-моему, ускорить принятие патчей в основное ядро поможет предварительный отбор для рассмотрения. На сайте RMDC есть люди, которые уже давно известны своими работами. Если сделать еще одну категорию, например, Проверенные патчи, куда будут переноситься работы, которые уже готовы к внесению в основную ветку. Чтобы Vladimir'у и другим пользователям с правами внесения в ядро уже не нужно было проверять заведомо хаковые патчи. Конечно, работа по предварительному одобрению, требует времени. Да и в большинстве случаев каждому хочется сделать свое решение, а не разбираться в чужом.

[Laise]

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

По-моему, ускорить принятие патчей в основное ядро поможет предварительный отбор для рассмотрения. На сайте RMDC есть люди, которые уже давно известны своими работами. Если сделать еще одну категорию, например, Проверенные патчи, куда будут переноситься работы, которые уже готовы к внесению в основную ветку. Чтобы Vladimir'у и другим пользователям с правами внесения в ядро уже не нужно было проверять заведомо хаковые патчи. Конечно, работа по предварительному одобрению, требует времени. Да и в большинстве случаев каждому хочется сделать свое решение, а не разбираться в чужом.

еще б не плохо нормальный формат патчей как на getmangos.com

[MaS0n]

Ну во-первых здесь не сложение аур, а сложение модификаторов, т.е численных значений аур, что вполне допустимо, см. код
Во-вторых, внимательно прочитать пост Владимира и я с ним согласен на 100%.
Надо проверять другие спеллы с этой аурой, нельзя по одному спеллу судить об ауре.
В типовом наименовании вообще о флай ничего не говорится, я встречал эту ауру и на нелетающих вехиклах

[KiriX]

Ну раз уж зашло обсуждение патча, который "сваял" (не иначе) я, то я позволю себе прокомментировать:
1) Странно, что я не поднял вопрос о том, что он до сих пор не принят (Если у кого возникают сомнения - я не xmolex, это не мой дубль и т.д и т.п.)
2) Патч писался (положа руку на сердце) на скорую руку лично для себя (в названии тема это отражено в скобках)
3) Сложение аур... В общем совершенно верный ответ дал MaS0n - такая ситуация действительно вполне возможна, поэтому я не стал убирать старый модификатор (GetTotalAuraModifier(SPELL_AURA_MOD_SPEED_FLIGHT)), а просто добавил в старый обработчик новый тип модификатор нового типа ауры
4) На момент написания патча не было клёвой проги - SpellWork, чтобы понять есть ли другие спеллы с такой аурой. Позже появилась возможность проверки, я её проводил, насколько помню - все спеллы подходят под данную реализацию. Сейчас нужно пересмотреть, но см. запись в моём дневнике...
5) Частично согласен с автором, действительно зачастую не хватает комментариев разработчиков, почему не принимается тот или иной патч. Лично для меня это не критично - все мы люди и понимаем объёмы работ разработчиков...

P.S: ХЗ когда смогу самостоятельно что-то просмотреть... И если уж вы указываете патч чей-то, как-то более прилично было бы сразу указать автора