Показать сообщение отдельно
Старый 30.11.2010, 18:01   #3
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от virusav Посмотреть сообщение
Код:
Player* Unit::GetCharmerOrOwnerPlayerOrPlayerItself()
...
return GetTypeId()==TYPEID_PLAYER ? (Player*)this : NULL;
Мне кажется, что не нужно
Код:
pKiller->GetTypeId() == TYPEID_PLAYER
Ну в общем-то да, хотя там проверка иная, моба может убить другой моб, но на нём врядли будет аура с нужного итема. А вообще первая строка никак не влияет на то, что попадёт в if, поэтому перестраховался.
Можно убирать - я сам тестил без этой строки - эту проверку только для патча добавил.
Хотя скорее эту проверку следовало бы вынести самой первой...
KiriX вне форума   Ответить с цитированием