База данных IDA от void17

Скрины лучше отправьте...

AlexSpl писал(а):

Цитата:

That's my point. As I said, moving about 90 MB of data every time among each other is annoying. Server is the best decision.

А как распределять время работы над базой? Одновременно же не поработаешь. Оптимально постить какие-то изменения отдельно и в случае их одобрения после бурного обсуждения участниками проекта вносить в базу.

Код: Выделить всё

if ( (_BYTE)shoot )
{
  if ( this->SpellDuration[AIR_SHIELD] == SUMMON_BOAT )
    goto LABEL_7;
  AirShield_Multiplier = this->AirShield_Multiplier;
}
else
{
  if ( this->SpellDuration[SHIELD] == SUMMON_BOAT )
    goto LABEL_7;
  AirShield_Multiplier = this->Shield_Multiplier;
}

Как избавиться от перечислителя? SUMMON_BOAT -> 0? Правый клик -> Decimal не работает, можно выбрать только значение из другого enum. Признаюсь, никогда не читал мануал по IDA (не было необходимости), но когда понимаешь, что не можешь исправить такие простейшие вещи, хочется его покурить

Как превратить это (нажал Show Casts, так легче понимать, что происходит):

Код: Выделить всё

if ( *(char *)(*((_DWORD *)&gpCombatManager->Hero1 + v17) + 221) > 1 )
      v42 = v42 + v42;

в gpCombatManager->Hero[Side]->second_skill[ARTILLERY]? Если в подсчётах не ошибся.

For the former, I have not found a solution ever sadly. IDA likes to apply enums with a heavy hand like butter.
Can you give me the address of the second snippet?

А вы меняли что? тип поля SpellDuration? Почему-то я поставил его тип SpellID, но enum для смещений в квадратных скобках не работает..(а жаль)

sub @ 0x442770. public: double army::get_unit_combat_value(long,long,bool,army const *)const proc near. Maybe it's because of two different fields Hero1 and Hero2 instead of array Hero[2]?

Нет, я уже открыл базу из бэкапа (чистую). В Вашей базе уже SUMMON_BOAT вместо 0.

Всё, допёр Для SpellDuration выставлен тип SpellID вместо int.

No. It's not their fault. Hero1 and Hero2 in my opinion are easier to read yet this is the same as Hero[2] but looks more original since in the original code there were two separated "class hero&" params.

Yes, use Hero[2].
Also a neat trick sometimes is T[1] to trick IDA in showing proper decompilation.

И получается мегакрасиво, как я привык и практикую:

Код: Выделить всё

if ( shoot )
{
  if ( !this->SpellDuration[AIR_SHIELD] )
    goto LABEL_7;
  AirShield_Multiplier = this->AirShield_Multiplier;
}
else
{
  if ( !this->SpellDuration[SHIELD] )
    goto LABEL_7;
  AirShield_Multiplier = this->Shield_Multiplier;
}

Кстати наши мнения насчет совместной работы над базой разошлись. Я и RoseKavalier считаем, что следует создать сервер и на нем работать. Как будем выходить из спора?

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

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

Also a chat room