Changeset 1ab8539 in mainline for uspace/app/bnchmark/bnchmark.c

Timestamp:

2014-11-14T19:16:23Z (9 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

5042706

Parents:

9696b01

Message:

remove system.uptime sysinfo entry since it is redundant
cleanup the time handling routines

• still more cleanup is needed here, the code is a wild combination of quasi-POSIX API (with all the brain-dead types, etc.) and HelenOS API

File:

• uspace/app/bnchmark/bnchmark.c (modified) (2 diffs)

uspace/app/bnchmark/bnchmark.c

@@ -62 +62 @@
 {
         struct timeval start_time;
-        int rc;
-        rc = gettimeofday(&start_time, NULL);
-        if (rc != EOK) {
-                fprintf(stderr, "gettimeofday failed\n");
-                return rc;
-        }
-        
-        rc = fn(data);
+        gettimeofday(&start_time, NULL);
+        
+        int rc = fn(data);
         if (rc != EOK) {
                 fprintf(stderr, "measured function failed\n");
                 return rc;
         }
-
+        
         struct timeval final_time;
-        rc = gettimeofday(&final_time, NULL);
-        if (rc != EOK) {
-                fprintf(stderr, "gettimeofday failed\n");
-                return rc;
-        }
+        gettimeofday(&final_time, NULL);
         
         /* Calculate time difference in milliseconds */
@@ -92 +83 @@
         char *path = (char *) data;
         char *buf = malloc(BUFSIZE);
-
-        if (buf == NULL) {
+        
+        if (buf == NULL)
                 return ENOMEM;
-        }
         
         FILE *file = fopen(path, "r");