diff --git a/src/server/distrend.c b/src/server/distrend.c --- a/src/server/distrend.c +++ b/src/server/distrend.c @@ -307,7 +307,7 @@ void frame_num_struct_builder(struct ble int frame_finder(struct blendjob *head, struct blendjob **job, struct frameset **frame) { int your_frame; // your_frame is an integer value that will be given to the client as the frame number to render - int your_job; // you_job is an integer value that must ALSO be given to the client + // UNUSED: int your_job; // @TODO: Fixme: do we need this var? you_job is an integer value that must ALSO be given to the client unsigned short int found; unsigned short int priority; @@ -512,7 +512,6 @@ int main(int argc, char *argv[]) { struct distren_action *action; int clientsays; /*< temporary example variable, will be replaced when we can handle messages */ - int framenumprevious; /*< @TODO why is this variable used? */ distrend_accept(&action); cont = distrend_do(action); @@ -548,7 +547,8 @@ int main(int argc, char *argv[]) /* If the client states that they finished the frame */ if(clientsays == DISTREN_REQUEST_DONEFRAME){ clientstatus = CLIENTSTATUS_IDLE; // Sets the client back to idle - finish_frame(jobnum, framenumprevious); // make it finish the previous frame or something, why framenumprevios? Is that the whole linked list thing coming in? + // finish_frame(jobnum, framenumprevious); // make it finish the previous frame or something, why framenumprevios? Is that the whole linked list thing coming in? + finish_frame(head, jobnum); // @TODO: update so it fits the purpose of the previous line } /* End Somewhat Pseudo-code */