*/
const struct AFB_interface *afbitf;
+/*
+ * definition of waiters
+ */
+struct waiter
+{
+ struct waiter *next;
+ struct afb_req req;
+};
+
/*
* definition of a board
*/
struct board
{
struct board *next;
- int refcount;
+ int use_count;
int moves;
int history[9];
int id;
int level;
char board[9];
+ struct waiter *waiters;
};
/*
static struct board *all_boards;
/*
- * Search a board
+ * Searchs a board having the 'id'.
+ * Returns it if found or NULL otherwise.
*/
static struct board *search_board(int id)
{
}
/*
- * Creates a new board
+ * Creates a new board and returns it.
*/
static struct board *get_new_board()
{
+ /* allocation */
struct board *board = calloc(1, sizeof *board);
+
+ /* initialisation */
memset(board->board, ' ', sizeof board->board);
- board->refcount = 1;
+ board->use_count = 1;
board->level = 1;
board->moves = 0;
do {
board->id = (rand() >> 2) % 1000;
} while(board->id == 0 || search_board(board->id) != NULL);
+
+ /* link */
board->next = all_boards;
all_boards = board;
return board;
static void release_board(struct board *board)
{
/* decrease the reference count ... */
- if (--board->refcount == 0) {
+ if (--board->use_count == 0) {
/* ... no more use */
/* unlink from the list of boards */
struct board **prv = &all_boards;
*/
static void changed(struct board *board, const char *reason)
{
- /* TODO */
- WARNING(afbitf, "changed is unimplmented (reason was %s)", reason);
+ struct waiter *waiter, *next;
+ struct json_object *description;
+
+ /* get the description */
+ description = describe(board);
+
+ waiter = board->waiters;
+ board->waiters = NULL;
+ while (waiter != NULL) {
+ next = waiter->next;
+ afb_req_success(waiter->req, json_object_get(description), reason);
+ afb_req_unref(waiter->req);
+ free(waiter);
+ waiter = next;
+ }
+
+ afb_daemon_broadcast_event(afbitf->daemon, reason, description);
}
/*
* retrieves the board of the request
*/
-static struct board *board_of_req(struct afb_req req)
+static inline struct board *board_of_req(struct afb_req req)
{
return afb_req_context(req, (void*)get_new_board, (void*)release_board);
}
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'new' called for boardid %d", board->id);
/* reset the game */
memset(board->board, ' ', sizeof board->board);
*/
static void board(struct afb_req req)
{
- afb_req_success(req, describe(board_of_req(req)), NULL);
+ struct board *board;
+ struct json_object *description;
+
+ /* retrieves the context for the session */
+ board = board_of_req(req);
+ INFO(afbitf, "method 'board' called for boardid %d", board->id);
+
+ /* describe the board */
+ description = describe(board);
+
+ /* send the board's description */
+ afb_req_success(req, description, NULL);
}
/*
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'move' called for boardid %d", board->id);
- /* retrieves the parameters of the move */
+ /* retrieves the arguments of the move */
index = afb_req_value(req, "index");
i = index == NULL ? -1 : atoi(index);
- /* checks validity of parameters */
+ /* checks validity of arguments */
if (i < 0 || i > 8) {
+ WARNING(afbitf, "can't move to %s: %s", index?:"?", index?"wrong value":"not set");
afb_req_fail(req, "error", "bad request");
return;
}
/* checks validity of the state */
if (winner(board->board) != 0) {
+ WARNING(afbitf, "can't move to %s: game is terminated", index);
afb_req_fail(req, "error", "game terminated");
return;
}
/* checks validity of the move */
if (board->board[i] != ' ') {
+ WARNING(afbitf, "can't move to %s: room occupied", index);
afb_req_fail(req, "error", "occupied");
return;
}
/* applies the move */
+ INFO(afbitf, "method 'move' for boardid %d, index=%s", board->id, index);
add_move(board, i);
/* replies */
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'level' called for boardid %d", board->id);
- /* retrieves the parameters */
+ /* retrieves the arguments */
level = afb_req_value(req, "level");
l = level == NULL ? -1 : atoi(level);
- /* check validity of parameters */
+ /* check validity of arguments */
if (l < 1 || l > 10) {
+ WARNING(afbitf, "can't set level to %s: %s", level?:"?", level?"wrong value":"not set");
afb_req_fail(req, "error", "bad request");
return;
}
+
+ /* set the level */
+ INFO(afbitf, "method 'level' for boardid %d, level=%d", board->id, l);
board->level = l;
/* replies */
*/
static void join(struct afb_req req)
{
- struct board *board;
+ struct board *board, *new_board;
const char *id;
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'join' called for boardid %d", board->id);
- /* retrieves the parameters */
- board = board_of_req(req);
+ /* retrieves the arguments */
id = afb_req_value(req, "boardid");
if (id == NULL)
goto bad_request;
- /* check validity of parameters */
- if (strcmp(id, "none")) {
- board = get_new_board();
+ /* none is a special id for joining a new session */
+ if (strcmp(id, "none") == 0) {
+ new_board = get_new_board();
goto success;
}
- board = search_board(atoi(id));
- if (board == NULL)
+
+ /* searchs the board to join */
+ new_board = search_board(atoi(id));
+ if (new_board == NULL)
goto bad_request;
- board->refcount++;
+ /*
+ * joining its board is stupid but possible
+ * however when called with the same stored pointer
+ * afb_req_context_set will not call the release
+ * function 'release_board'. So the use_count MUST not
+ * be incremented.
+ */
+ if (new_board != board)
+ new_board->use_count++;
+
success:
- afb_req_context_set(req, board, (void*)release_board);
+ /* set the new board (and leaves the previous one) */
+ afb_req_context_set(req, new_board, (void*)release_board);
/* replies */
afb_req_success(req, NULL, NULL);
return;
bad_request:
+ WARNING(afbitf, "can't join boardid %s: %s", id ? : "?", !id ? "no boardid" : atoi(id) ? "not found" : "bad boardid");
afb_req_fail(req, "error", "bad request");
return;
}
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'undo' called for boardid %d", board->id);
/* checks the state */
if (board->moves == 0) {
+ WARNING(afbitf, "can't undo");
afb_req_fail(req, "error", "bad request");
return;
}
/* retrieves the context for the session */
board = board_of_req(req);
+ INFO(afbitf, "method 'play' called for boardid %d", board->id);
/* checks validity of the state */
if (winner(board->board) != 0 || board->moves == 9) {
+ WARNING(afbitf, "can't play: game terminated (%s)", winner(board->board) ? "has winner" : "no room left");
afb_req_fail(req, "error", "game terminated");
return;
}
changed(board, "play");
}
+static void wait(struct afb_req req)
+{
+ int count;
+ struct board *board;
+ struct waiter *waiter;
+
+ /* retrieves the context for the session */
+ board = board_of_req(req);
+ INFO(afbitf, "method 'wait' called for boardid %d", board->id);
+
+ /* counts the waiters */
+ count = 0;
+ waiter = board->waiters;
+ while (waiter != NULL) {
+ count++;
+ waiter = waiter->next;
+ }
+
+ /* checks ability to wait */
+ if (count + 1 >= board->use_count) {
+ WARNING(afbitf, "can't wait: count=%d and use_count=%d", count, board->use_count);
+ afb_req_fail(req, "error", "can't wait");
+ return;
+ }
+
+ /* creates the waiter and enqueues it */
+ waiter = calloc(1, sizeof *waiter);
+ waiter->req = req;
+ waiter->next = board->waiters;
+ afb_req_addref(req);
+ board->waiters = waiter;
+}
+
/*
* array of the verbs exported to afb-daemon
*/
-static const struct AFB_verb_desc_v1 verbs[] = {
+static const struct AFB_verb_desc_v1 plugin_verbs[] = {
/* VERB'S NAME SESSION MANAGEMENT FUNCTION TO CALL SHORT DESCRIPTION */
{ .name= "new", .session= AFB_SESSION_NONE, .callback= new, .info= "Starts a new game" },
- { .name= "play", .session= AFB_SESSION_NONE, .callback= play, .info= "Tells the server to play" },
+ { .name= "play", .session= AFB_SESSION_NONE, .callback= play, .info= "Asks the server to play" },
{ .name= "move", .session= AFB_SESSION_NONE, .callback= move, .info= "Tells the client move" },
{ .name= "board", .session= AFB_SESSION_NONE, .callback= board, .info= "Get the current board" },
{ .name= "level", .session= AFB_SESSION_NONE, .callback= level, .info= "Set the server level" },
- { .name= "join", .session= AFB_SESSION_NONE, .callback= join, .info= "Join a board" },
+ { .name= "join", .session= AFB_SESSION_CHECK,.callback= join, .info= "Join a board" },
{ .name= "undo", .session= AFB_SESSION_NONE, .callback= undo, .info= "Undo the last move" },
- /* marker for end of the array */
- { .name= NULL }
+ { .name= "wait", .session= AFB_SESSION_NONE, .callback= wait, .info= "Wait for a change" },
+ { .name= NULL } /* marker for end of the array */
};
/*
.v1= { /* fills the v1 field of the union when AFB_PLUGIN_VERSION_1 */
.prefix= "tictactoe", /* the API name (or plugin name or prefix) */
.info= "Sample tac-tac-toe game", /* short description of of the plugin */
- .verbs = verbs /* the array describing the verbs of the API */
+ .verbs = plugin_verbs /* the array describing the verbs of the API */
}
};
Code Review / src / app-framework-binder.git / blobdiff