From bf7f722d0f42b37a83df91d0a62544c948c4cb66 Mon Sep 17 00:00:00 2001
From: Jan Breuer <jan.breuer@jaybee.cz>
Date: 周日, 24 4月 2016 20:24:11 +0800
Subject: [PATCH] Extend travis script to test all configurations with device dependent error
---
libscpi/src/error.c | 123 +++++++++++++++++------------------------
1 files changed, 51 insertions(+), 72 deletions(-)
diff --git a/libscpi/src/error.c b/libscpi/src/error.c
index 7b17f4c..e07ced3 100644
--- a/libscpi/src/error.c
+++ b/libscpi/src/error.c
@@ -41,6 +41,12 @@
#include "scpi/error.h"
#include "fifo_private.h"
+#if USE_DEVICE_DEPENDENT_ERROR_INFORMATION
+#define SCPI_ERROR_SETVAL(e, c, i) do { (e)->error_code = (c); (e)->device_dependent_info = (i); } while(0)
+#else
+#define SCPI_ERROR_SETVAL(e, c, i) do { (e)->error_code = (c); } while(0)
+#endif
+
/**
* Initialize error queue
* @param context - scpi context
@@ -82,11 +88,11 @@
*/
void SCPI_ErrorClear(scpi_t * context) {
#if USE_DEVICE_DEPENDENT_ERROR_INFORMATION
- scpi_error_t error;
- while(fifo_remove(&context->error_queue, &error)){
- SCPIDEFINE_free(&context->error_info_heap, error.device_dependent_info, false);
- }
-#endif
+ scpi_error_t error;
+ while (fifo_remove(&context->error_queue, &error)) {
+ SCPIDEFINE_free(&context->error_info_heap, error.device_dependent_info, false);
+ }
+#endif
fifo_clear(&context->error_queue);
SCPI_ErrorEmitEmpty(context);
@@ -96,31 +102,16 @@
* Pop error from queue
* @param context - scpi context
* @param error
- * @return
+ * @return
*/
-scpi_bool_t SCPI_ErrorPopEx(scpi_t * context, scpi_error_t * error) {
- if(!error) return FALSE;
- error->error_code = 0;
- error->device_dependent_info = NULL;
- fifo_remove(&context->error_queue, error);
-
- SCPI_ErrorEmitEmpty(context);
-
- return TRUE;
-}
-/**
- * Pop error from queue
- * @param context - scpi context
- * @return error number
- */
-int16_t SCPI_ErrorPop(scpi_t * context) {
- int16_t result = 0;
-
- fifo_remove(&context->error_queue, &result);
+scpi_bool_t SCPI_ErrorPop(scpi_t * context, scpi_error_t * error) {
+ if (!error || !context) return FALSE;
+ SCPI_ERROR_SETVAL(error, 0, NULL);
+ fifo_remove(&context->error_queue, error);
SCPI_ErrorEmitEmpty(context);
- return result;
+ return TRUE;
}
/**
@@ -137,9 +128,13 @@
}
static scpi_bool_t SCPI_ErrorAddInternal(scpi_t * context, int16_t err, char * info) {
- if (!fifo_add(&context->error_queue, err, info)) {
- fifo_remove_last(&context->error_queue, NULL);
- fifo_add(&context->error_queue, SCPI_ERROR_QUEUE_OVERFLOW, NULL);
+ scpi_error_t error_value;
+ SCPI_ERROR_SETVAL(&error_value, err, info);
+ if (!fifo_add(&context->error_queue, &error_value)) {
+ fifo_remove_last(&context->error_queue, &error_value);
+ SCPIDEFINE_free(&context->error_info_heap, error_value.device_dependent_info, false);
+ SCPI_ERROR_SETVAL(&error_value, SCPI_ERROR_QUEUE_OVERFLOW, NULL);
+ fifo_add(&context->error_queue, &error_value);
return FALSE;
}
return TRUE;
@@ -148,10 +143,10 @@
struct error_reg {
int16_t from;
int16_t to;
- scpi_reg_val_t bit;
+ scpi_reg_val_t esrBit;
};
-#define ERROR_DEFS_N 9
+#define ERROR_DEFS_N 9
static const struct error_reg errs[ERROR_DEFS_N] = {
{-100, -199, ESR_CER}, /* Command error (e.g. syntax error) ch 21.8.9 */
@@ -170,33 +165,35 @@
* @param context - scpi context
* @param err - error number
*/
-void SCPI_ErrorPushEx(scpi_t * context, int16_t err, char * info) {
- int i;
- char * info_ptr = NULL;
-
+void SCPI_ErrorPushEx(scpi_t * context, int16_t err, char * info, size_t info_len) {
+ int i;
+ char * info_ptr = NULL;
+
#if USE_DEVICE_DEPENDENT_ERROR_INFORMATION
- if (info){
- info_ptr = SCPIDEFINE_strdup(&context->error_info_heap, info);
- }
+ if (info) {
+ info_ptr = SCPIDEFINE_strndup(&context->error_info_heap, info, info_len);
+ }
#endif
- scpi_bool_t queue_overflow = !SCPI_ErrorAddInternal(context, err, info_ptr);
+ scpi_bool_t queue_overflow = !SCPI_ErrorAddInternal(context, err, info_ptr);
- for (i = 0; i < ERROR_DEFS_N; i++) {
- if ((err <= errs[i].from) && (err >= errs[i].to)) {
- SCPI_RegSetBits(context, SCPI_REG_ESR, errs[i].bit);
- }
- }
+ for (i = 0; i < ERROR_DEFS_N; i++) {
+ if ((err <= errs[i].from) && (err >= errs[i].to)) {
+ SCPI_RegSetBits(context, SCPI_REG_ESR, errs[i].esrBit);
+ }
+ }
- SCPI_ErrorEmit(context, err);
- if (queue_overflow) {
- SCPI_ErrorEmit(context, SCPI_ERROR_QUEUE_OVERFLOW);
- SCPIDEFINE_free(&context->error_info_heap, info_ptr, true);
- }
+ SCPI_ErrorEmit(context, err);
+ if (queue_overflow) {
+ SCPI_ErrorEmit(context, SCPI_ERROR_QUEUE_OVERFLOW);
+#if USE_DEVICE_DEPENDENT_ERROR_INFORMATION
+ SCPIDEFINE_free(&context->error_info_heap, info_ptr, true);
+#endif
+ }
- if (context) {
- context->cmd_error = TRUE;
- }
+ if (context) {
+ context->cmd_error = TRUE;
+ }
}
/**
@@ -205,26 +202,8 @@
* @param err - error number
*/
void SCPI_ErrorPush(scpi_t * context, int16_t err) {
- SCPI_ErrorPushEx(context, err, NULL);
- return;
- //int i;
-//
- //scpi_bool_t queue_overflow = !SCPI_ErrorAddInternal(context, err, NULL);
-//
- //for (i = 0; i < ERROR_DEFS_N; i++) {
- //if ((err <= errs[i].from) && (err >= errs[i].to)) {
- //SCPI_RegSetBits(context, SCPI_REG_ESR, errs[i].bit);
- //}
- //}
-//
- //SCPI_ErrorEmit(context, err);
- //if (queue_overflow) {
- //SCPI_ErrorEmit(context, SCPI_ERROR_QUEUE_OVERFLOW);
- //}
-//
- //if (context) {
- //context->cmd_error = TRUE;
- //}
+ SCPI_ErrorPushEx(context, err, NULL, 0);
+ return;
}
/**
--
Gitblit v1.9.1