Changeset 20546

Timestamp:

Nov 5, 2008, 12:58:04 AM (18 years ago)

Author:

Sebastian Jester

Message:

Ignore return value of write in a way that makes compiler happy and avoids avoid compiler warning: "ignoring return value of write, declared with attribute warn_unused_result" - the compiler was too clever for the fix in 1.71 of psLogMsg.c

Location:

trunk/psLib/src/sys

Files:

• psLogMsg.c (modified) (3 diffs)
• psTrace.c (modified) (3 diffs)

trunk/psLib/src/sys/psLogMsg.c

@@ -11 +11 @@
  *  @author GLG, MHPCC
  *
- *  @version $Revision: 1.71 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2008-11-04 18:36:54 $
+ *  @version $Revision: 1.72 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2008-11-05 10:58:04 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -315 +315 @@
     *head_ptr = '\0';
 
-    (void)write(logFD, head, strlen(head));
+    if (write(logFD, head, strlen(head))) {;} // ignore return value
+    
     if (logMsg) {
         psString msg = NULL;            // Message to print
@@ -324 +325 @@
         char *line = strtok_r(msg, "\n", &msgPtr);
         while (line) {
-            (void)write(logFD, "    ", 4);
-            (void)write(logFD, line, strlen(line));
-            (void)write(logFD, "\n", 1);
+            if(write(logFD, "    ", 4)) {;} // ignore return value
+            if(write(logFD, line, strlen(line))) {;} // ignore return value
+            if(write(logFD, "\n", 1)) {;} // ignore return value
             line = strtok_r(NULL, "\n", &msgPtr);
         }
         psFree(msg);
     } else {
-        (void)write(logFD, "\n", 1);
+        if(write(logFD, "\n", 1)) {;} // ignore return value
     }
 

trunk/psLib/src/sys/psTrace.c

@@ -9 +9 @@
  *  @author GLG, MHPCC
  *
- *  @version $Revision: 1.87 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2008-04-17 23:43:02 $
+ *  @version $Revision: 1.88 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2008-11-05 10:58:04 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -486 +486 @@
     } else {
         if (comp->level == PS_DEFAULT_TRACE_LEVEL) {
-            sprintf(line,"%*s%-*s %d\n", depth, "", 20 - depth, comp->name, defLevel);
-            write (traceFD, line, strlen(line));
+            sprintf(line,"%*s%-*s %d\n", depth, "", 20 - depth, comp->name, defLevel);
+            if (write (traceFD, line, strlen(line))) {;} // ignore return value
         } else {
-            sprintf(line, "%*s%-*s %d\n", depth, "", 20 - depth, comp->name, comp->level);
-            write (traceFD, line, strlen(line));
+            sprintf(line, "%*s%-*s %d\n", depth, "", 20 - depth, comp->name, comp->level);
+            if (write (traceFD, line, strlen(line))) {;} // ignore return value
         }
     }
@@ -615 +615 @@
         *head_ptr = '\0';
 
-        write(traceFD, head, strlen(head));
+        if(write(traceFD, head, strlen(head)) ) {;} // ignore return value
 
         if (traceMsg) {
             // We indent each message one space for each level of the message.
             for (int i = 0; i < level; i++) {
-                write (traceFD, " ", 1);
+                if (write (traceFD, " ", 1)) {;} // ignore return value
             }
             psString line = NULL;       // Line to print
             psStringAppendV(&line, format, ap);
-            write (traceFD, line, strlen(line));
+            if (write (traceFD, line, strlen(line))) {;} // ignore return value
             if (line[strlen(line) - 1] != '\n') {
-                write(traceFD, "\n", 1);
+                if (write(traceFD, "\n", 1)) {;} // ignore return value
             }
             psFree(line);
         } else {
-            write(traceFD, "\n", 1);
+            if (write(traceFD, "\n", 1)) {;} // ignore return value
         }