comparison vamp-server/convert.cpp @ 180:bd543e74a9bf

Correct the inspection of the JSON object in successful to look for both error and success objects, writing out an error string if neither are present. Revert error handling in readInput() for JSON.

     eof = false;
     
     if (format == "json") {
         string err;
         auto result = readInputJson(direction, err, eof);
-        const bool isRecognisedError = !result.success && result.errorText != "";
-        
-        // if the RequestOrResponse (result) has been populated with success=false and an error message
-        // then the server returned a well formed error, it is safe to return it for conversion 
-        // -- but if err is populated, something else has gone wrong
-        if (isRecognisedError || err == "") 
-            return result;
-        else 
-            throw runtime_error(err);
+        if (err != "") throw runtime_error(err);
+        else return result;
     } else if (format == "capnp") {
         return readInputCapnp(direction, eof);
     } else {
         throw runtime_error("unknown input format \"" + format + "\"");
     }