TARGET_MASTER

[virusav]

В формировании списка целей:

        case TARGET_MASTER:
            if(Unit* owner = m_caster->GetCharmerOrOwner())
                targetUnitMap.push_back(owner);

Интересно, почему GetCharmerOrOwner() берется именно от кастера, а не от таргета.

Пример: у игрока есть квестовый пет, после убийства определенного нпц надо кастовать спелл с TARGET_MASTER.
Варианты:
1. Пет кастует спелл киллкредита на себя, в результате чего игрок получает +1 (текущая реализация).
2. НПЦ в момент смерти кастует спелл на пета, в результате чего игрок получает +1.

Как правильно?

[Vladimir]

Просто не было примеров спелов, которые не для кастера?

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

Т.е . в вашем случае надо смотреть есть ли примеры спелов, где это требуется реально.