From bdfb235a97f69d82538449a033e3e5f1637365d0 Mon Sep 17 00:00:00 2001
From: Christopher Ferris <cferris@google.com>
Date: Fri, 3 May 2013 12:51:45 -0700
Subject: Fix a few incorrect return codes.

Merge from internal master.

(cherry-picked from f90ab5f4ab00ddfbcf313be9001837aaefd64ba2)

Change-Id: I87379bac089107a95301015d849433c9e304a988
---
 tests/memtest/bandwidth.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/memtest/bandwidth.cpp b/tests/memtest/bandwidth.cpp
index 0e0ec2e0..523cabe3 100644
--- a/tests/memtest/bandwidth.cpp
+++ b/tests/memtest/bandwidth.cpp
@@ -306,7 +306,7 @@ int per_core_bandwidth(int argc, char** argv) {
         args[i].core = *it;
         args[i].bench = createBandwidthBenchmarkObject(values);
         if (!args[i].bench) {
-            return 0;
+            return -1;
         }
     }
 
@@ -342,7 +342,7 @@ int multithread_bandwidth(int argc, char** argv) {
         args[i].core = -1;
         args[i].bench = createBandwidthBenchmarkObject(values);
         if (!args[i].bench) {
-            return 0;
+            return -1;
         }
     }
 
@@ -367,18 +367,18 @@ bool run_bandwidth_benchmark(int argc, char** argv, const char *name,
     values["num_warm_loops"].int_value = 0;
     values["num_loops"].int_value = 0;
     if (!processBandwidthOptions(argc, argv, bandwidth_opts, &values)) {
-        return -1;
+        return false;
     }
 
     size_t size = values["size"].int_value;
     if ((size % 64) != 0) {
         printf("The size value must be a multiple of 64.\n");
-        return 1;
+        return false;
     }
 
     if (setpriority(PRIO_PROCESS, 0, -20)) {
         perror("Unable to raise priority of process.");
-        return -1;
+        return false;
     }
 
     bool preamble_printed = false;
-- 
cgit v1.2.3