Skip to content

Change ZoneClient and ChatClient to public access#1887

Open
AtmoOmen wants to merge 1 commit into
aers:mainfrom
AtmoOmen:patch-4
Open

Change ZoneClient and ChatClient to public access#1887
AtmoOmen wants to merge 1 commit into
aers:mainfrom
AtmoOmen:patch-4

Conversation

@AtmoOmen

Copy link
Copy Markdown
Contributor

given that the structures of ZoneClient and ChatClient have been largely reverse-engineered, making them public enables support for plugins such as those for latency testing.

@pohky

pohky commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator

yeah but they are wrong so that needs to be fixed first

@AtmoOmen

Copy link
Copy Markdown
Contributor Author

yeah but they are wrong so that needs to be fixed first

at least HostPort in ClientBase look fine.
I could make another pr later, but I think it's better to make them public first. After all, they are not totally wrong.

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