Changeset 33985 for branches/eam_branches/ipp-20120601/Ohana/src/opihi/pantasks/ControllerOps.c

Timestamp:

Jun 6, 2012, 9:16:29 PM (14 years ago)

Author:

eugene

Message:

gcc 4.6.3 adds a new level of pedantry: if a variable is defined and set, but not used, it raises a warning. with -Werror, this forces a lot of minor fixes. I do not think any of the resultings changes caught any real problems, at least not with any commonly used code

File:

• branches/eam_branches/ipp-20120601/Ohana/src/opihi/pantasks/ControllerOps.c (modified) (6 diffs)

branches/eam_branches/ipp-20120601/Ohana/src/opihi/pantasks/ControllerOps.c

@@ -63 +63 @@
 int CheckControllerJob (Job *job) {
   struct timeval start, stop;
-  float dtime;
 
   gettimeofday (&start, (void *) NULL);
   CheckControllerJobStatus (job);
   gettimeofday (&stop, (void *) NULL);
-  dtime = DTIME (stop, start);
-  /* if (VerboseMode()) gprint (GP_ERR, "check job status %f\n", dtime); */
+  // float dtime = DTIME (stop, start);
+  // if (VerboseMode()) gprint (GP_ERR, "check job status %f\n", dtime);
 
   if ((job[0].state == JOB_EXIT) || (job[0].state == JOB_CRASH)) {
@@ -75 +74 @@
     GetJobOutput ("stdout", job[0].pid, &job[0].stdout_buff, job[0].stdout_size);
     gettimeofday (&stop, (void *) NULL);
-    dtime = DTIME (stop, start);
+    // float dtime = DTIME (stop, start);
     /* if (VerboseMode()) gprint (GP_ERR, "get stdout %f\n", dtime); */
 
@@ -81 +80 @@
     GetJobOutput ("stderr", job[0].pid, &job[0].stderr_buff, job[0].stderr_size);
     gettimeofday (&stop, (void *) NULL);
-    dtime = DTIME (stop, start);
+    // float dtime = DTIME (stop, start);
     /* if (VerboseMode()) gprint (GP_ERR, "get stderr %f\n", dtime); */
 
@@ -87 +86 @@
     DeleteControllerJob (job);
     gettimeofday (&stop, (void *) NULL);
-    dtime = DTIME (stop, start);
+    // float dtime = DTIME (stop, start);
     /* if (VerboseMode()) gprint (GP_ERR, "delete job %f\n", dtime); */
   }  
@@ -578 +577 @@
 int QuitController () {
 
-  int status;
   char cmd[128];
   IOBuffer buffer;
@@ -586 +584 @@
   sprintf (cmd, "quit");
   InitIOBuffer (&buffer, 0x100);
-  status = ControllerCommand (cmd, NULL, &buffer);
+  ControllerCommand (cmd, NULL, &buffer);
   FreeIOBuffer (&buffer);