11-22-2012, 09:27 AM
ОМГ. Не, ну серьёзно, почему "упоротые" и "нуба взяли на работу"? Вы никогда не задумывались, как именно у них отражены опкоды пакетов в сорцах? Скорее всего, это сделано в виде enum-а в общем для клиента и сервера header файле.
enum PacketsExtend
{
FIRST_PACKET = 0,
SECOND_PACKET, // 1
THIRD_PACKET, // 2
// ...
PLEDGE_SKILL_LIST_PACKET, // 0x3A
PLEDGE_SKILL_ADD_PACKET, // 0x3B
PLEDGE_SKILL_REMOVE_PACKET, // 0x3C вставлен, всё что ниже просто сдвинулось на единицу
// ...
BEFORE_THE_LAST_PACKET, // 0xFE, был 0xFD
LAST_PACKET, // 0xFF, был 0xFE
},
И если раньше, они скорее всего просто добавляли опкоды просто добавлением строчки в этот enum, то сейчас появилась более чем логичная идея сгруппировать пакеты с одним смыслом вместе. Понятное дело, что всё сдвинулось!
Быть может, я покажусь капитаном очевидностью, но блин, если такие комментарии на эту тему появляются, то ничего, кроме удивления, у меня это не вызывает.
И то, что у GW-шников всё быстро поправлено, МБ есть следствие того, что они перепилили свой движок именно под использовании подобных enum-ов?
enum PacketsExtend
{
FIRST_PACKET = 0,
SECOND_PACKET, // 1
THIRD_PACKET, // 2
// ...
PLEDGE_SKILL_LIST_PACKET, // 0x3A
PLEDGE_SKILL_ADD_PACKET, // 0x3B
PLEDGE_SKILL_REMOVE_PACKET, // 0x3C вставлен, всё что ниже просто сдвинулось на единицу
// ...
BEFORE_THE_LAST_PACKET, // 0xFE, был 0xFD
LAST_PACKET, // 0xFF, был 0xFE
},
И если раньше, они скорее всего просто добавляли опкоды просто добавлением строчки в этот enum, то сейчас появилась более чем логичная идея сгруппировать пакеты с одним смыслом вместе. Понятное дело, что всё сдвинулось!
Быть может, я покажусь капитаном очевидностью, но блин, если такие комментарии на эту тему появляются, то ничего, кроме удивления, у меня это не вызывает.
И то, что у GW-шников всё быстро поправлено, МБ есть следствие того, что они перепилили свой движок именно под использовании подобных enum-ов?