DistRen Changeset - 80a8b6d4c92a · protofusion repositories

Changeset - 80a8b6d4c92a

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

ethanzonca - 16 years ago 2009-08-01 00:22:57

We are now warning free! Goodnight.

1 file changed with 3 insertions and 3 deletions:

src/server/distrend.c

0 comments (0 inline, 0 general)

src/server/distrend.c

➞

Show inline comments

@@ @@ -304,13 +304,13 @@ void frame_num_struct_builder(struct ble @@
   @return 0 success, other: error
 */
 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;
   struct blendjob *blendjob_ptr;
@@ @@ -509,13 +509,12 @@ int main(int argc, char *argv[]) @@
   distrend_listen(&listenset, config);
   /* This is called the ``main loop'' */
   while(cont)
+    {
       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);
       /* Somewhat Pseudo-code for basic server operation, should be more event-driven */
@@ @@ -545,13 +544,14 @@ int main(int argc, char *argv[]) @@
 	    remotio_send_to_client(frame->num, job->jobnum); // Did you actually make this function, ohnobinki? --normaldotcom
+	}
       /* 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 */
       distrend_action_free(action);

0 comments (0 inline, 0 general)