ThePhrozenKeep/D2MOO

Have a clean way to declare and reference packet tables

Open

#100 ouverte le 23 janv. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (48 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (159 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Stack technique
Domaine
Type d'issue
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Prérequis
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.