Skip to content

Fix m_szServerName and m_szClientName length#21

Merged
catornot merged 4 commits into
R2NorthstarTools:masterfrom
AllusiveWheat:master
Mar 30, 2026
Merged

Fix m_szServerName and m_szClientName length#21
catornot merged 4 commits into
R2NorthstarTools:masterfrom
AllusiveWheat:master

Conversation

@AllusiveWheat

Copy link
Copy Markdown
Contributor

makes them 256 like they should be

@catornot catornot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❯ cargo fmt --check
Diff in /home/catornot/dev/rust/rrplug/src/bindings/class_types/client.rs:1:

-use std::ffi::{ c_char, c_void };
+use std::ffi::{c_char, c_void};

-use crate::{ field_assert, size_assert };
+use crate::{field_assert, size_assert};

 const PERSISTENCE_MAX_SIZE: usize = 0xddcd;

Diff in /home/catornot/dev/rust/rrplug/src/bindings/class_types/client.rs:64:

 pub enum SignonState {
     #[default]
     NONE = 0, // no state yet; about to connect
-    CHALLENGE = 1, // client challenging server; all OOB packets
-    CONNECTED = 2, // client is connected to server; netchans ready
-    NEW = 3, // just got serverinfo and string tables
-    PRESPAWN = 4, // received signon buffers
+    CHALLENGE = 1,   // client challenging server; all OOB packets
+    CONNECTED = 2,   // client is connected to server; netchans ready
+    NEW = 3,         // just got serverinfo and string tables
+    PRESPAWN = 4,    // received signon buffers
     GETTINGDATA = 5, // respawn-defined signonstate, assumedly this is for persistence
-    SPAWN = 6, // ready to receive entity packets
-    FIRSTSNAP = 7, // another respawn-defined one
-    FULL = 8, // we are fully connected; first non-delta packet received
+    SPAWN = 6,       // ready to receive entity packets
+    FIRSTSNAP = 7,   // another respawn-defined one
+    FULL = 8,        // we are fully connected; first non-delta packet received
     CHANGELEVEL = 9, // server is changing level; please wait
 }

@catornot catornot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve

@catornot catornot merged commit d8e5b32 into R2NorthstarTools:master Mar 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants