aubinator: be consistent on exit code
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Wed, 24 May 2017 01:02:30 +0000 (02:02 +0100)
committerLionel Landwerlin <lionel.g.landwerlin@intel.com>
Wed, 24 May 2017 09:50:18 +0000 (10:50 +0100)
We're using both exit(1) & exit(EXIT_FAILURE), settle for one, same
for success.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
src/intel/tools/aubinator_error_decode.c

index 506d39012b869fd82efacf03160f5ca77a44da88..a41292238cb7ef2731d8b5f78cb9abde10c2b50a 100644 (file)
@@ -587,7 +587,7 @@ read_data_file(FILE *file)
          count = ascii85_decode(line+1, &data, line[0] == ':');
          if (count == 0) {
             fprintf(stderr, "ASCII85 decode failed.\n");
-            exit(1);
+            exit(EXIT_FAILURE);
          }
 
          if (strcmp(buffer_name, "user") == 0) {
@@ -718,7 +718,7 @@ read_data_file(FILE *file)
          data = realloc(data, data_size * sizeof (uint32_t));
          if (data == NULL) {
             fprintf(stderr, "Out of memory.\n");
-            exit(1);
+            exit(EXIT_FAILURE);
          }
       }
 
@@ -824,7 +824,7 @@ main(int argc, char *argv[])
 
    if (help || argc == 1) {
       print_help(argv[0], stderr);
-      exit(0);
+      exit(EXIT_SUCCESS);
    }
 
    if (optind >= argc) {
@@ -847,7 +847,7 @@ main(int argc, char *argv[])
          }
       } else {
          read_data_file(stdin);
-         exit(0);
+         exit(EXIT_SUCCESS);
       }
    } else {
       path = argv[optind];
@@ -855,7 +855,7 @@ main(int argc, char *argv[])
       if (error != 0) {
          fprintf(stderr, "Error opening %s: %s\n",
                  path, strerror(errno));
-         exit(1);
+         exit(EXIT_FAILURE);
       }
    }