Throw, instead of asserting for Coverity Scan

Since the latter won't help in release mode. This is still test code only. Otherwise, we'd do something a little more formal.
badone · Apr 8, 2015 · bcfa65c · bcfa65c
1 parent 48a418b
commit bcfa65c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/test/unit/src/fake_libpcp.cpp b/test/unit/src/fake_libpcp.cpp
@@ -67,8 +67,10 @@ void __pmNotifyErr(int /*priority*/, const char */*message*/, ...)
 
 int __pmParseDebug(const char *spec)
 {
-    // The first this the real __pmParseDebug does is dereference spec.
-    assert(spec != NULL);
+    // The first thing the real __pmParseDebug does is dereference spec.
+    if (spec == NULL) {
+        throw "spec must not be NULL";
+    }
 
     // Allow our unit tests to invoke an error response.
     if (strcmp(spec, "invalid") == 0) {