From af4e630a29e8ddcb11f377d3b0e6f6d580040313 Mon Sep 17 00:00:00 2001
From: Jan Breuer <jan.breuer@jaybee.cz>
Date: 周五, 12 12月 2014 17:50:29 +0800
Subject: [PATCH] Parser refactoring

---
 libscpi/src/units.c |   67 +++++++++++++++++++++------------
 1 files changed, 42 insertions(+), 25 deletions(-)

diff --git a/libscpi/src/units.c b/libscpi/src/units.c
index 3583ef5..2b646e9 100644
--- a/libscpi/src/units.c
+++ b/libscpi/src/units.c
@@ -37,9 +37,9 @@
 #include <string.h>
 #include "scpi/parser.h"
 #include "scpi/units.h"
-#include "utils.h"
+#include "utils_private.h"
 #include "scpi/error.h"
-#include "scpi/lexer.h"
+#include "lexer_private.h"
 
 
 /*
@@ -110,7 +110,7 @@
     {/* name */ "UP", /* type */ SCPI_NUM_UP},
     {/* name */ "DOWN", /* type */ SCPI_NUM_DOWN},
     {/* name */ "NAN", /* type */ SCPI_NUM_NAN},
-    {/* name */ "INF", /* type */ SCPI_NUM_INF},
+    {/* name */ "INFinity", /* type */ SCPI_NUM_INF},
     {/* name */ "NINF", /* type */ SCPI_NUM_NINF},
     SCPI_SPECIAL_NUMBERS_LIST_END,
 };
@@ -123,7 +123,7 @@
  * @param value resultin value
  * @return TRUE if str matches one of specs patterns
  */
-static bool_t translateSpecialNumber(const scpi_special_number_def_t * specs, const char * str, size_t len, scpi_number_parameter_t * value) {
+static scpi_bool_t translateSpecialNumber(const scpi_special_number_def_t * specs, const char * str, size_t len, scpi_number_t * value) {
     int i;
 
     value->value = 0.0;
@@ -219,7 +219,7 @@
  * @param value preparsed numeric value
  * @return TRUE if value parameter was converted to base units
  */
-static bool_t transformNumber(scpi_t * context, const char * unit, size_t len, scpi_number_parameter_t * value) {
+static scpi_bool_t transformNumber(scpi_t * context, const char * unit, size_t len, scpi_number_t * value) {
     size_t s;
     const scpi_unit_def_t * unitDef;
     s = skipWhitespace(unit, len);
@@ -249,33 +249,50 @@
  * @param mandatory if the parameter is mandatory
  * @return 
  */
-bool_t SCPI_ParamTranslateNumberVal(scpi_t * context, scpi_parameter_t * parameter) {
-    token_t token;
+scpi_bool_t SCPI_ParamNumber(scpi_t * context, scpi_number_t * value, scpi_bool_t mandatory)
+{
+    scpi_token_t token;
     lex_state_t state;
+    scpi_parameter_t param;
+    scpi_bool_t result;
+    
+    result = SCPI_Parameter(context, &param, mandatory);
+    
 
-    state.buffer = parameter->data.ptr;
+    state.buffer = param.data.ptr;
     state.pos = state.buffer;
-    state.len = parameter->data.len;
+    state.len = param.data.len;
 
-    if (parameter->type == TokDecimalNumericProgramDataWithSuffix) {
-        SCPI_LexDecimalNumericProgramData(&state, &token);
-        SCPI_LexWhiteSpace(&state, &token);
-        SCPI_LexSuffixProgramData(&state, &token);
+    switch(param.type) {
+        case TokDecimalNumericProgramData:
+        case TokHexnum:
+        case TokOctnum:
+        case TokBinnum:
+            result = TRUE;
+            break;
+        case TokDecimalNumericProgramDataWithSuffix:
+            scpiLex_DecimalNumericProgramData(&state, &token);
+            scpiLex_WhiteSpace(&state, &token);
+            scpiLex_SuffixProgramData(&state, &token);
 
-        return transformNumber(context, token.ptr, token.len, &parameter->number);
+            result = transformNumber(context, token.ptr, token.len, &param.number);
+            break;
+        case TokProgramMnemonic:
+            scpiLex_WhiteSpace(&state, &token);
+            scpiLex_CharacterProgramData(&state, &token);
 
-    } else if (parameter->type == TokProgramMnemonic) {
-        SCPI_LexWhiteSpace(&state, &token);
-        SCPI_LexCharacterProgramData(&state, &token);
-
-        /* convert string to special number type */
-        if (translateSpecialNumber(context->special_numbers, token.ptr, token.len, &parameter->number)) {
-            /* found special type */
-            return TRUE;
-        }
+            /* convert string to special number type */
+            result = translateSpecialNumber(context->special_numbers, token.ptr, token.len, &param.number);
+            break;
+        default:
+            result = FALSE;
     }
 
-    return FALSE;
+    if (result) {
+        memcpy(value, &param.number, sizeof(scpi_number_t));
+    }
+
+    return result;
 }
 
 /**
@@ -286,7 +303,7 @@
  * @param len max length of string
  * @return number of chars written to string
  */
-size_t SCPI_NumberToStr(scpi_t * context, scpi_number_parameter_t * value, char * str, size_t len) {
+size_t SCPI_NumberToStr(scpi_t * context, scpi_number_t * value, char * str, size_t len) {
     const char * type;
     const char * unit;
     size_t result;

--
Gitblit v1.9.1