ThePhrozenKeep/D2MOO

Have a clean way to declare and reference packet tables

Open

#100 aperta il 23 gen 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (48 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (159 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In D2Net Client.cpp, the size of the packet sizes table (std::size(gServerPacketSizeTable)) is hardcoded (0xB4).

We should probably have a constant for this, or put the table in its own header since it should only be included in D2Net's Client.cpp and Server.cpp.

Originally posted by @Lectem in https://github.com/ThePhrozenKeep/D2MOO/pull/96#discussion_r1083280497

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.