Skip to content
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

Регистрация JSON-комманд #40

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Регистрация JSON-комманд #40

wants to merge 6 commits into from
+634 −426

Conversation

ghost
Copy link

@ghost ghost commented Feb 9, 2016

Нужно еще дописать обработку JSON исключений

@edapin
Move some classes to separate files for future common use
19b8621 
Move
	genRandomString to MyUtils
	JsonRequest, JsonResponse, JsonException to JSONRequestHandler.hpp
@edapin
replace #prama once with include guards according to style guidelines
103452f
@edapin
Move code from GameHandler.hpp to GameHandler.cpp
f69d10c 
Now GameHandler.hpp contains interface only.
@edapin
add Websocket.receiveFrame exception handling
7f38d96 
WebsocketReceive returns number of bytes received. A return value of 0
means that connection is closed, -1 means that exception occured.
@edapin
change buffer data structure from string to list of strings
cb20a4c
@edapin
register json commands that can be received by websocket
ad85ad3
@davidyuk
Copy link
Collaborator

davidyuk commented Feb 9, 2016

Я думаю, MyUtils нужно переименовать, например, в Utils или Common.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidyuk @edapin