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.

Sign up for GitHub

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

Jump to bottom

WebSockets: Use better variable names. #187

Open

andywolk wants to merge 1 commit into master from vocabulary

Contributor

andywolk commented Jan 9, 2024

No description provided.


          WebSockets: Use better variable names.

d31ab9b

andywolk requested a review from crienzo

January 9, 2024 22:22

Contributor

seven1240 commented Jan 9, 2024

there's a lot of short vars, I think a comment on the struct would just do it? Or doc it in doxygen format.

Contributor Author

andywolk commented Jan 9, 2024

These vars are very confusing when you read the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet