-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(net): attribute proper packet names for timeout #2141
feat(net): attribute proper packet names for timeout #2141
Conversation
Looks good, but while ur at it, maybe add the hash as a // comment so we can ctrl-f for it sometime later |
5d4b521
to
c269245
Compare
c269245
to
b11e635
Compare
Applied the requested changes, also replaced g_knownPackets in NetDebug.cpp to use g_knownPackets from KnownPackets.h |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 more things that stand out
- Can you move
KnownPackets.h
in a subfolder callednet
and put it in namespacefx::net
? - See comment below
b11e635
to
7800517
Compare
if a player times out instead of getting the packet hash they will get the packet name
7800517
to
eff3edb
Compare
This pretty simple PR has been sitting for 9 months, so I'm going to assume the changes are unwanted. |
This makes it so if a player times out, instead of getting packet hash as hex they'll get the actual packet name