You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Mirror #191: two new create-time error codes and the unchecked callback class #7
The C reference landed a robustness batch in mas-bandwidth/netcode#191 (from netcode#190). Two pieces of it are mirrored surface in this port and should land here too. No wire format change in either.
1. Two new create-time error codes, mirrored by value
The enums here mirror those values position for position — Client.cs:41-56 and Server.cs:21-36 — so the two new codes must land in the same positions, appended after the existing last member of each, not renumbered around.
2. The unchecked callback class
The same null-callback class exists in this port:
Client.cs:496
Client.cs:729
Server.cs:881
In C the fix is a create-time refusal: if override_send_and_receive is set and either override callback is null, create fails with the new code rather than dereferencing the pointer on the first update; and the loopback entry points refuse a loopback client or slot when the loopback send callback is unset, in every build. The same refusal should happen here, at the same points, so a misconfigured caller gets a failed create instead of a null dereference.
Citing netcode#191 for the reasoning and the red/green on each.
Moved to the roadmap. Since 2026-10-14 the planned work for this repo lives in docs/roadmap.sexp, rendered as ROADMAP.md, and the entry for this item ("Mirror #191: two new create-time error codes and the unchecked callback class") is in main since #11 merged today. Closing the issue so the board counts only what the roadmap does not already hold; the work is still owed there.
Activity
rowan-claude commentedon Oct 16, 2026
Moved to the roadmap. Since 2026-10-14 the planned work for this repo lives in
docs/roadmap.sexp, rendered asROADMAP.md, and the entry for this item ("Mirror #191: two new create-time error codes and the unchecked callback class") is in main since #11 merged today. Closing the issue so the board counts only what the roadmap does not already hold; the work is still owed there.