X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=error.c;h=a04a983fc8908313af6c13a9becd2b36745bf0de;hb=99c76642aa8d3c75bef4ef3927b4c4cf45461892;hp=ce1b0401ddbad2a8dc2aeb54d4d62335ec2db31d;hpb=50985c2019b2b5a6ce394589d89ee925b4f5e3a9;p=libsigrok.git

diff --git a/error.c b/error.c
index ce1b0401..a04a983f 100644
--- a/error.c
+++ b/error.c
@@ -52,41 +52,35 @@
  */
 SR_API const char *sr_strerror(int error_code)
 {
-	const char *str;
-
 	/*
-	 * Note: All defined SR_* error macros from libsigrok.h should have
+	 * Note: All defined SR_* error macros from libsigrok.h must have
 	 * an entry in this function, as well as in sr_strerror_name().
 	 */
 
 	switch (error_code) {
 	case SR_OK:
-		str = "no error";
-		break;
+		return "no error";
 	case SR_ERR:
-		str = "generic/unspecified error";
-		break;
+		return "generic/unspecified error";
 	case SR_ERR_MALLOC:
-		str = "memory allocation error";
-		break;
+		return "memory allocation error";
 	case SR_ERR_ARG:
-		str = "invalid argument";
-		break;
+		return "invalid argument";
 	case SR_ERR_BUG:
-		str = "internal error";
-		break;
+		return "internal error";
 	case SR_ERR_SAMPLERATE:
-		str = "invalid samplerate";
-		break;
+		return "invalid samplerate";
 	case SR_ERR_NA:
-		str = "not applicable";
-		break;
+		return "not applicable";
+	case SR_ERR_DEV_CLOSED:
+		return "device closed but should be open";
+	case SR_ERR_TIMEOUT:
+		return "timeout occurred";
+	case SR_ERR_PROBE_GROUP:
+		return "no probe group specified";
 	default:
-		str = "unknown error";
-		break;
+		return "unknown error";
 	}
-
-	return str;
 }
 
 /**
@@ -109,41 +103,35 @@ SR_API const char *sr_strerror(int error_code)
  */
 SR_API const char *sr_strerror_name(int error_code)
 {
-	const char *str;
-
 	/*
-	 * Note: All defined SR_* error macros from libsigrok.h should have
+	 * Note: All defined SR_* error macros from libsigrok.h must have
 	 * an entry in this function, as well as in sr_strerror().
 	 */
 
 	switch (error_code) {
 	case SR_OK:
-		str = "SR_OK";
-		break;
+		return "SR_OK";
 	case SR_ERR:
-		str = "SR_ERR";
-		break;
+		return "SR_ERR";
 	case SR_ERR_MALLOC:
-		str = "SR_ERR_MALLOC";
-		break;
+		return "SR_ERR_MALLOC";
 	case SR_ERR_ARG:
-		str = "SR_ERR_ARG";
-		break;
+		return "SR_ERR_ARG";
 	case SR_ERR_BUG:
-		str = "SR_ERR_BUG";
-		break;
+		return "SR_ERR_BUG";
 	case SR_ERR_SAMPLERATE:
-		str = "SR_ERR_SAMPLERATE";
-		break;
+		return "SR_ERR_SAMPLERATE";
 	case SR_ERR_NA:
-		str = "SR_ERR_NA";
-		break;
+		return "SR_ERR_NA";
+	case SR_ERR_DEV_CLOSED:
+		return "SR_ERR_DEV_CLOSED";
+	case SR_ERR_TIMEOUT:
+		return "SR_ERR_TIMEOUT";
+	case SR_ERR_PROBE_GROUP:
+		return "SR_PROBE_GROUP";
 	default:
-		str = "unknown error code";
-		break;
+		return "unknown error code";
 	}
-
-	return str;
 }
 
 /** @} */