From 11f2f2e329ef404d0e9c022cb2f9fbbb45bae285 Mon Sep 17 00:00:00 2001
From: nancy.liao <huihui.liao@greentest.com.cn>
Date: 周日, 27 4月 2025 17:33:31 +0800
Subject: [PATCH] 完成了SCPI命令语法分析器的完整规则
---
examples/common/scpi-def.c | 239 ++++++++++++++++++++++++++++++-----------------------------
1 files changed, 123 insertions(+), 116 deletions(-)
diff --git a/examples/common/scpi-def.c b/examples/common/scpi-def.c
index 809fb1e..d46c067 100644
--- a/examples/common/scpi-def.c
+++ b/examples/common/scpi-def.c
@@ -1,37 +1,38 @@
/*-
- * Copyright (c) 2012-2013 Jan Breuer,
+ * BSD 2-Clause License
*
- * All Rights Reserved
- *
+ * Copyright (c) 2012-2018, Jan Breuer
+ * All rights reserved.
+ *
* Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- * 1. Redistributions of source code must retain the above copyright notice,
- * this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHORS ``AS IS'' AND ANY EXPRESS OR
- * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
- * DISCLAIMED. IN NO EVENT SHALL THE AUTHORS OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
- * BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
- * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
- * OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
- * IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * * Redistributions of source code must retain the above copyright notice, this
+ * list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright notice,
+ * this list of conditions and the following disclaimer in the documentation
+ * and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
/**
* @file scpi-def.c
* @date Thu Nov 15 10:58:45 UTC 2012
- *
+ *
* @brief SCPI parser test
- *
- *
+ *
+ *
*/
#include <stdio.h>
@@ -43,16 +44,16 @@
static scpi_result_t DMM_MeasureVoltageDcQ(scpi_t * context) {
scpi_number_t param1, param2;
char bf[15];
- fprintf(stderr, "meas:volt:dc\r\n"); // debug command name
+ fprintf(stderr, "meas:volt:dc\r\n"); /* debug command name */
- // read first parameter if present
+ /* read first parameter if present */
if (!SCPI_ParamNumber(context, scpi_special_numbers_def, ¶m1, FALSE)) {
- // do something, if parameter not present
+ /* do something, if parameter not present */
}
- // read second paraeter if present
+ /* read second paraeter if present */
if (!SCPI_ParamNumber(context, scpi_special_numbers_def, ¶m2, FALSE)) {
- // do something, if parameter not present
+ /* do something, if parameter not present */
}
@@ -71,16 +72,16 @@
static scpi_result_t DMM_MeasureVoltageAcQ(scpi_t * context) {
scpi_number_t param1, param2;
char bf[15];
- fprintf(stderr, "meas:volt:ac\r\n"); // debug command name
+ fprintf(stderr, "meas:volt:ac\r\n"); /* debug command name */
- // read first parameter if present
+ /* read first parameter if present */
if (!SCPI_ParamNumber(context, scpi_special_numbers_def, ¶m1, FALSE)) {
- // do something, if parameter not present
+ /* do something, if parameter not present */
}
- // read second paraeter if present
+ /* read second paraeter if present */
if (!SCPI_ParamNumber(context, scpi_special_numbers_def, ¶m2, FALSE)) {
- // do something, if parameter not present
+ /* do something, if parameter not present */
}
@@ -98,16 +99,16 @@
static scpi_result_t DMM_ConfigureVoltageDc(scpi_t * context) {
double param1, param2;
- fprintf(stderr, "conf:volt:dc\r\n"); // debug command name
+ fprintf(stderr, "conf:volt:dc\r\n"); /* debug command name */
- // read first parameter if present
+ /* read first parameter if present */
if (!SCPI_ParamDouble(context, ¶m1, TRUE)) {
return SCPI_RES_ERR;
}
- // read second paraeter if present
+ /* read second paraeter if present */
if (!SCPI_ParamDouble(context, ¶m2, FALSE)) {
- // do something, if parameter not present
+ /* do something, if parameter not present */
}
fprintf(stderr, "\tP1=%lf\r\n", param1);
@@ -118,9 +119,9 @@
static scpi_result_t TEST_Bool(scpi_t * context) {
scpi_bool_t param1;
- fprintf(stderr, "TEST:BOOL\r\n"); // debug command name
+ fprintf(stderr, "TEST:BOOL\r\n"); /* debug command name */
- // read first parameter if present
+ /* read first parameter if present */
if (!SCPI_ParamBool(context, ¶m1, TRUE)) {
return SCPI_RES_ERR;
}
@@ -181,9 +182,9 @@
const char * data;
size_t len;
- SCPI_ParamArbitraryBlock(context, &data, &len, FALSE);
-
- SCPI_ResultArbitraryBlock(context, data, len);
+ if (SCPI_ParamArbitraryBlock(context, &data, &len, FALSE)) {
+ SCPI_ResultArbitraryBlock(context, data, len);
+ }
return SCPI_RES_OK;
}
@@ -195,130 +196,137 @@
typedef struct _scpi_channel_value_t scpi_channel_value_t;
/**
- * @brief
+ * @brief
* parses lists
* channel numbers > 0.
* no checks yet.
* valid: (@1), (@3!1:1!3), ...
* (@1!1:3!2) would be 1!1, 1!2, 2!1, 2!2, 3!1, 3!2.
* (@3!1:1!3) would be 3!1, 3!2, 3!3, 2!1, 2!2, 2!3, ... 1!3.
- *
+ *
* @param channel_list channel list, compare to SCPI99 Vol 1 Ch. 8.3.2
*/
static scpi_result_t TEST_Chanlst(scpi_t *context) {
scpi_parameter_t channel_list_param;
-#define maxrow 2 //maximum number of rows
-#define maxcol 6 //maximum number of columns
-#define maxdim 2 //maximum number of dimensions
- scpi_channel_value_t array[maxrow * maxcol]; //array which holds values in order (2D)
- size_t chanlst_idx; //index for channel list
- size_t arr_idx = 0; //index for array
- size_t n, m = 1; //counters for row (n) and columns (m)
+#define MAXROW 2 /* maximum number of rows */
+#define MAXCOL 6 /* maximum number of columns */
+#define MAXDIM 2 /* maximum number of dimensions */
+ scpi_channel_value_t array[MAXROW * MAXCOL]; /* array which holds values in order (2D) */
+ size_t chanlst_idx; /* index for channel list */
+ size_t arr_idx = 0; /* index for array */
+ size_t n, m = 1; /* counters for row (n) and columns (m) */
- // get channel list
+ /* get channel list */
if (SCPI_Parameter(context, &channel_list_param, TRUE)) {
scpi_expr_result_t res;
scpi_bool_t is_range;
- int32_t values_from[maxdim];
- int32_t values_to[maxdim];
+ int32_t values_from[MAXDIM];
+ int32_t values_to[MAXDIM];
size_t dimensions;
- bool for_stop_row = false; //true if iteration for rows has to stop
- bool for_stop_col = false; //true if iteration for columns has to stop
- int32_t dir_row = 1; //direction of counter for rows, +/-1
- int32_t dir_col = 1; //direction of counter for columns, +/-1
+ bool for_stop_row = FALSE; /* true if iteration for rows has to stop */
+ bool for_stop_col = FALSE; /* true if iteration for columns has to stop */
+ int32_t dir_row = 1; /* direction of counter for rows, +/-1 */
+ int32_t dir_col = 1; /* direction of counter for columns, +/-1 */
- // the next statement is valid usage and it gets only real number of dimensions for the first item (index 0)
+ /* the next statement is valid usage and it gets only real number of dimensions for the first item (index 0) */
if (!SCPI_ExprChannelListEntry(context, &channel_list_param, 0, &is_range, NULL, NULL, 0, &dimensions)) {
- chanlst_idx = 0; //call first index
- arr_idx = 0; //set arr_idx to 0
- do { //if valid, iterate over channel_list_param index while res == valid (do-while cause we have to do it once)
+ chanlst_idx = 0; /* call first index */
+ arr_idx = 0; /* set arr_idx to 0 */
+ do { /* if valid, iterate over channel_list_param index while res == valid (do-while cause we have to do it once) */
res = SCPI_ExprChannelListEntry(context, &channel_list_param, chanlst_idx, &is_range, values_from, values_to, 4, &dimensions);
- if (is_range == false) { //still can have multiple dimensions
+ if (is_range == FALSE) { /* still can have multiple dimensions */
if (dimensions == 1) {
- //here we have our values
- //row == values_from[0]
- //col == 0 (fixed number)
- //call a function or something
+ /* here we have our values
+ * row == values_from[0]
+ * col == 0 (fixed number)
+ * call a function or something */
array[arr_idx].row = values_from[0];
array[arr_idx].col = 0;
} else if (dimensions == 2) {
- //here we have our values
- //row == values_fom[0]
- //col == values_from[1]
- //call a function or something
+ /* here we have our values
+ * row == values_fom[0]
+ * col == values_from[1]
+ * call a function or something */
array[arr_idx].row = values_from[0];
array[arr_idx].col = values_from[1];
} else {
return SCPI_RES_ERR;
- break;
}
- arr_idx++; //inkrement array where we want to save our values to, not neccessary otherwise
- } else if (is_range == true) {
+ arr_idx++; /* inkrement array where we want to save our values to, not neccessary otherwise */
+ if (arr_idx >= MAXROW * MAXCOL) {
+ return SCPI_RES_ERR;
+ }
+ } else if (is_range == TRUE) {
if (values_from[0] > values_to[0]) {
- dir_row = -1; //we have to decrement from values_from
- } else { //if (values_from[0] < values_to[0])
- dir_row = +1; //default, we increment from values_from
+ dir_row = -1; /* we have to decrement from values_from */
+ } else { /* if (values_from[0] < values_to[0]) */
+ dir_row = +1; /* default, we increment from values_from */
}
- //iterating over rows, do it once -> set for_stop_row = false
- //needed if there is channel list index isn't at end yet
- for_stop_row = false;
- for (n = values_from[0]; for_stop_row == false; n += dir_row) {
- //usual case for ranges, 2 dimensions
+ /* iterating over rows, do it once -> set for_stop_row = false
+ * needed if there is channel list index isn't at end yet */
+ for_stop_row = FALSE;
+ for (n = values_from[0]; for_stop_row == FALSE; n += dir_row) {
+ /* usual case for ranges, 2 dimensions */
if (dimensions == 2) {
if (values_from[1] > values_to[1]) {
dir_col = -1;
} else if (values_from[1] < values_to[1]) {
dir_col = +1;
}
- //iterating over columns, do it at least once -> set for_stop_col = false
- //needed if there is channel list index isn't at end yet
- for_stop_col = false;
- for (m = values_from[1]; for_stop_col == false; m += dir_col) {
- //here we have our values
- //row == n
- //col == m
- //call a function or something
+ /* iterating over columns, do it at least once -> set for_stop_col = false
+ * needed if there is channel list index isn't at end yet */
+ for_stop_col = FALSE;
+ for (m = values_from[1]; for_stop_col == FALSE; m += dir_col) {
+ /* here we have our values
+ * row == n
+ * col == m
+ * call a function or something */
array[arr_idx].row = n;
array[arr_idx].col = m;
arr_idx++;
+ if (arr_idx >= MAXROW * MAXCOL) {
+ return SCPI_RES_ERR;
+ }
if (m == (size_t)values_to[1]) {
- //endpoint reached, stop column for-loop
- for_stop_col = true;
+ /* endpoint reached, stop column for-loop */
+ for_stop_col = TRUE;
}
}
- //special case for range, example: (@2!1)
+ /* special case for range, example: (@2!1) */
} else if (dimensions == 1) {
- //here we have values
- //row == n
- //col == 0 (fixed number)
- //call function or sth.
+ /* here we have values
+ * row == n
+ * col == 0 (fixed number)
+ * call function or sth. */
array[arr_idx].row = n;
array[arr_idx].col = 0;
arr_idx++;
+ if (arr_idx >= MAXROW * MAXCOL) {
+ return SCPI_RES_ERR;
+ }
}
if (n == (size_t)values_to[0]) {
- //endpoint reached, stop row for-loop
- for_stop_row = true;
+ /* endpoint reached, stop row for-loop */
+ for_stop_row = TRUE;
}
}
} else {
return SCPI_RES_ERR;
- break;
}
- //increase index
+ /* increase index */
chanlst_idx++;
} while (SCPI_EXPR_OK == SCPI_ExprChannelListEntry(context, &channel_list_param, chanlst_idx, &is_range, values_from, values_to, 4, &dimensions));
- //while checks, whether incremented index is valid
+ /* while checks, whether incremented index is valid */
}
- //do something at the end if needed
- array[arr_idx].row = 0;
- array[arr_idx].col = 0;
+ /* do something at the end if needed */
+ /* array[arr_idx].row = 0; */
+ /* array[arr_idx].col = 0; */
}
-
+
{
size_t i;
fprintf(stderr, "TEST_Chanlst: ");
@@ -366,14 +374,14 @@
{.pattern = "SYSTem:ERRor:COUNt?", .callback = SCPI_SystemErrorCountQ,},
{.pattern = "SYSTem:VERSion?", .callback = SCPI_SystemVersionQ,},
- //{.pattern = "STATus:OPERation?", .callback = scpi_stub_callback,},
- //{.pattern = "STATus:OPERation:EVENt?", .callback = scpi_stub_callback,},
- //{.pattern = "STATus:OPERation:CONDition?", .callback = scpi_stub_callback,},
- //{.pattern = "STATus:OPERation:ENABle", .callback = scpi_stub_callback,},
- //{.pattern = "STATus:OPERation:ENABle?", .callback = scpi_stub_callback,},
+ /* {.pattern = "STATus:OPERation?", .callback = scpi_stub_callback,}, */
+ /* {.pattern = "STATus:OPERation:EVENt?", .callback = scpi_stub_callback,}, */
+ /* {.pattern = "STATus:OPERation:CONDition?", .callback = scpi_stub_callback,}, */
+ /* {.pattern = "STATus:OPERation:ENABle", .callback = scpi_stub_callback,}, */
+ /* {.pattern = "STATus:OPERation:ENABle?", .callback = scpi_stub_callback,}, */
{.pattern = "STATus:QUEStionable[:EVENt]?", .callback = SCPI_StatusQuestionableEventQ,},
- //{.pattern = "STATus:QUEStionable:CONDition?", .callback = scpi_stub_callback,},
+ /* {.pattern = "STATus:QUEStionable:CONDition?", .callback = scpi_stub_callback,}, */
{.pattern = "STATus:QUEStionable:ENABle", .callback = SCPI_StatusQuestionableEnable,},
{.pattern = "STATus:QUEStionable:ENABle?", .callback = SCPI_StatusQuestionableEnableQ,},
@@ -399,7 +407,6 @@
{.pattern = "TEST:TEXT", .callback = TEST_Text,},
{.pattern = "TEST:ARBitrary?", .callback = TEST_ArbQ,},
{.pattern = "TEST:CHANnellist", .callback = TEST_Chanlst,},
-
SCPI_CMD_LIST_END
};
@@ -412,6 +419,6 @@
};
char scpi_input_buffer[SCPI_INPUT_BUFFER_LENGTH];
-int16_t scpi_error_queue_data[SCPI_ERROR_QUEUE_SIZE];
+scpi_error_t scpi_error_queue_data[SCPI_ERROR_QUEUE_SIZE];
scpi_t scpi_context;
--
Gitblit v1.9.1