Комментарии (14) RSS

• absolut 14 лет ago # 0

  что было во 2ой строке?

  Ответить

  • HaskellGovno 14 лет ago # 0 ⇈

    Раньше было так:
    

    uint32 originalDamage = damage;
    
    if (absorb && originalDamage > damage)
        *absorb += (originalDamage - damage);

    
    Сместилось имеенно из-за постинга

    Ответить

  • roman-kashitsyn 14 лет ago # 0 ⇈

    если хорошо приглядеться, можно увидеть \n

    Ответить

    • HaskellGovno 14 лет ago # 0 ⇈

      и \r

      Ответить

      • absolut 14 лет ago # 0 ⇈

        Это если под win

        Ответить

        • bormand 14 лет ago # 0 ⇈

          @HaskellGovno прав, в исходнике данной страницы стоят и \r и \n.

          Ответить

          • absolut 14 лет ago # 0 ⇈

            а ещё `&` и `>`

            Ответить

• Heisenberg 14 лет ago # 0

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

  Ответить

• guest 14 лет ago # 0

  Если вкопаться в TrinityCore по глубже, и не такой говнокод увидишь.

  Ответить

• TarasB 14 лет ago # 0

  где говно?

  Ответить

  • absolut 14 лет ago # 0 ⇈

    то есть присваивание с последующим сравнением тебя не смущают?

    Ответить

• sidsukana 14 лет ago # 0

  А ничего что вся функция выглядит вот так?
  

  void Unit::DealDamageMods(Unit *pVictim, uint32 &damage, uint32* absorb)
  {
      if (!pVictim->isAlive() || pVictim->IsTaxiFlying() || (pVictim->GetTypeId() == TYPEID_UNIT && ((Creature*)pVictim)->IsInEvadeMode()))
      {
          if (absorb)
              *absorb += damage;
          damage = 0;
          return;
      }
  
      uint32 originalDamage = damage;
  
      //Script Event damage Deal
      if (GetTypeId() == TYPEID_UNIT && ((Creature *)this)->AI())
          ((Creature *)this)->AI()->DamageDeal(pVictim, damage);
      //Script Event damage taken
      if (pVictim->GetTypeId() == TYPEID_UNIT && ((Creature *)pVictim)->AI())
          ((Creature *)pVictim)->AI()->DamageTaken(this, damage);
  
      if (absorb && originalDamage > damage)
          *absorb += (originalDamage - damage);
  }

  
  
  И переменные проверяются после модифицирования, так как они ссылочные. Автор не позорься.

  Ответить

• Heisenberg 14 лет ago # 0

  Не надо меня учить) Не знаю где вы взяли этот код, но в мастер ветке тринити (я брал его оттуда, увидел в коммите) такого и близко нету. Вот что там
  https://github.com/TrinityCore/TrinityCore/blob/4.3.4/src/server/game/Entities/Unit/Unit.cpp#L543
  вот после удаления
  
  а вот этот коммит https://github.com/TrinityCore/TrinityCore/commit/8eba4aebfb56bf86a74fc9841af75359b9cf2346

  Ответить

• sidsukana 14 лет ago # 0

  Тогда извиняюсь. Я код взял из мангоса и других его форков, там код выше присутствует вместе с вызовами виртуалок. Видимо тринити когда-то убрали скриптовые ивенты отсюда и забыли про мертвый код.

  Ответить