From 4d39507183e0ace2d73f04817a398e858b66b677 Mon Sep 17 00:00:00 2001
From: Jan Breuer <jan.breuer@jaybee.cz>
Date: 周五, 22 3月 2013 01:40:22 +0800
Subject: [PATCH] Update lexer, start of implementing parser

---
 libscpi/src/units.c |  160 ++++++++++++++++++++++++++++++++++------------------
 1 files changed, 104 insertions(+), 56 deletions(-)

diff --git a/libscpi/src/units.c b/libscpi/src/units.c
index a22f770..493c4ce 100644
--- a/libscpi/src/units.c
+++ b/libscpi/src/units.c
@@ -62,71 +62,93 @@
  */
 const scpi_unit_def_t scpi_units_def[] = {
     /* voltage */
-    { .name = "UV", .unit = SCPI_UNIT_VOLT, .mult = 1e-6},
-    { .name = "MV", .unit = SCPI_UNIT_VOLT, .mult = 1e-3},
-    { .name = "V", .unit = SCPI_UNIT_VOLT, .mult = 1},
-    { .name = "KV", .unit = SCPI_UNIT_VOLT, .mult = 1e3},
+    {/* name */ "UV",   /* unit */ SCPI_UNIT_VOLT,      /* mult */ 1e-6},
+    {/* name */ "MV",   /* unit */ SCPI_UNIT_VOLT,      /* mult */ 1e-3},
+    {/* name */ "V",    /* unit */ SCPI_UNIT_VOLT,      /* mult */ 1},
+    {/* name */ "KV",   /* unit */ SCPI_UNIT_VOLT,      /* mult */ 1e3},
 
     /* current */
-    { .name = "UA", .unit = SCPI_UNIT_AMPER, .mult = 1e-6},
-    { .name = "MA", .unit = SCPI_UNIT_AMPER, .mult = 1e-3},
-    { .name = "A", .unit = SCPI_UNIT_AMPER, .mult = 1},
-    { .name = "KA", .unit = SCPI_UNIT_AMPER, .mult = 1e3},
+    {/* name */ "UA",   /* unit */ SCPI_UNIT_AMPER,     /* mult */ 1e-6},
+    {/* name */ "MA",   /* unit */ SCPI_UNIT_AMPER,     /* mult */ 1e-3},
+    {/* name */ "A",    /* unit */ SCPI_UNIT_AMPER,     /* mult */ 1},
+    {/* name */ "KA",   /* unit */ SCPI_UNIT_AMPER,     /* mult */ 1e3},
 
     /* resistance */
-    { .name = "OHM", .unit = SCPI_UNIT_OHM, .mult = 1},
-    { .name = "KOHM", .unit = SCPI_UNIT_OHM, .mult = 1e3},
-    { .name = "MOHM", .unit = SCPI_UNIT_OHM, .mult = 1e6},
+    {/* name */ "OHM",  /* unit */ SCPI_UNIT_OHM,       /* mult */ 1},
+    {/* name */ "KOHM", /* unit */ SCPI_UNIT_OHM,       /* mult */ 1e3},
+    {/* name */ "MOHM", /* unit */ SCPI_UNIT_OHM,       /* mult */ 1e6},
 
     /* frequency */
-    { .name = "HZ", .unit = SCPI_UNIT_HERTZ, .mult = 1},
-    { .name = "KHZ", .unit = SCPI_UNIT_HERTZ, .mult = 1e3},
-    { .name = "MHZ", .unit = SCPI_UNIT_HERTZ, .mult = 1e6},
-    { .name = "GHZ", .unit = SCPI_UNIT_HERTZ, .mult = 1e9},
+    {/* name */ "HZ",   /* unit */ SCPI_UNIT_HERTZ,     /* mult */ 1},
+    {/* name */ "KHZ",  /* unit */ SCPI_UNIT_HERTZ,     /* mult */ 1e3},
+    {/* name */ "MHZ",  /* unit */ SCPI_UNIT_HERTZ,     /* mult */ 1e6},
+    {/* name */ "GHZ",  /* unit */ SCPI_UNIT_HERTZ,     /* mult */ 1e9},
 
     /* temperature */
-    { .name = "CEL", .unit = SCPI_UNIT_CELSIUS, .mult = 1},
+    {/* name */ "CEL",  /* unit */ SCPI_UNIT_CELSIUS,   /* mult */ 1},
 
     /* time */
-    { .name = "PS", .unit = SCPI_UNIT_SECONDS, .mult = 1e-12},
-    { .name = "NS", .unit = SCPI_UNIT_SECONDS, .mult = 1e-9},
-    { .name = "US", .unit = SCPI_UNIT_SECONDS, .mult = 1e-6},
-    { .name = "MS", .unit = SCPI_UNIT_SECONDS, .mult = 1e-3},
-    { .name = "S", .unit = SCPI_UNIT_SECONDS, .mult = 1},
-    { .name = "MIN", .unit = SCPI_UNIT_SECONDS, .mult = 60},
-    { .name = "HR", .unit = SCPI_UNIT_SECONDS, .mult = 3600},
+    {/* name */ "PS",   /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 1e-12},
+    {/* name */ "NS",   /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 1e-9},
+    {/* name */ "US",   /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 1e-6},
+    {/* name */ "MS",   /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 1e-3},
+    {/* name */ "S",    /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 1},
+    {/* name */ "MIN",  /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 60},
+    {/* name */ "HR",   /* unit */ SCPI_UNIT_SECONDS,   /* mult */ 3600},
 
     SCPI_UNITS_LIST_END,
 };
 
+/*
+ * Special number values definition
+ */
 const scpi_special_number_def_t scpi_special_numbers_def[] = {
-    { .name = "MINimum", .type = SCPI_NUM_MIN},
-    { .name = "MAXimum", .type = SCPI_NUM_MAX},
-    { .name = "DEFault", .type = SCPI_NUM_DEF},
-    { .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 = "NINF", .type = SCPI_NUM_NINF},
+    {/* name */ "MINimum",      /* type */ SCPI_NUM_MIN},
+    {/* name */ "MAXimum",      /* type */ SCPI_NUM_MAX},
+    {/* name */ "DEFault",      /* type */ SCPI_NUM_DEF},
+    {/* 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 */ "NINF",         /* type */ SCPI_NUM_NINF},
     SCPI_SPECIAL_NUMBERS_LIST_END,
 };
 
-static scpi_special_number_t translateSpecialNumber(const scpi_special_number_def_t * specs, const char * str, size_t len) {
+/**
+ * Match string constant to one of special number values
+ * @param specs specifications of special numbers (patterns)
+ * @param str string to be recognised
+ * @param len length of string
+ * @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_t * value) {
     int i;
 
+    value->value = 0.0;
+    value->unit = SCPI_UNIT_NONE;
+    value->type = SCPI_NUM_NUMBER;
+
     if (specs == NULL) {
-        return SCPI_NUM_NUMBER;
+        return FALSE;
     }
-    
+
     for (i = 0; specs[i].name != NULL; i++) {
         if (matchPattern(specs[i].name, strlen(specs[i].name), str, len)) {
-            return specs[i].type;
+            value->type = specs[i].type;
+            return TRUE;
         }
     }
 
-    return SCPI_NUM_NUMBER;
+    return FALSE;
 }
 
+/**
+ * Convert special number type to its string representation
+ * @param specs specifications of special numbers (patterns)
+ * @param type type of special number
+ * @return String representing special number or NULL
+ */
 static const char * translateSpecialNumberInverse(const scpi_special_number_def_t * specs, scpi_special_number_t type) {
     int i;
 
@@ -143,6 +165,13 @@
     return NULL;
 }
 
+/**
+ * Convert string describing unit to its representation
+ * @param units units patterns
+ * @param unit text representation of unknown unit
+ * @param len length of text representation
+ * @return pointer of related unit definition or NULL
+ */
 static const scpi_unit_def_t * translateUnit(const scpi_unit_def_t * units, const char * unit, size_t len) {
     int i;
     
@@ -159,6 +188,12 @@
     return NULL;
 }
 
+/**
+ * Convert unit definition to string
+ * @param units units definitions (patterns)
+ * @param unit type of unit
+ * @return string representation of unit
+ */
 static const char * translateUnitInverse(const scpi_unit_def_t * units, const scpi_unit_t unit) {
     int i;
     
@@ -175,7 +210,15 @@
     return NULL;
 }
 
-static bool_t transformNumber(const scpi_unit_def_t * units, const char * unit, size_t len, scpi_number_t * value) {
+/**
+ * Transform number to base units
+ * @param context
+ * @param unit text representation of unit
+ * @param len length of text representation
+ * @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_t * value) {
     size_t s;
     const scpi_unit_def_t * unitDef;
     s = skipWhitespace(unit, len);
@@ -185,9 +228,10 @@
         return TRUE;
     }
 
-    unitDef = translateUnit(units, unit + s, len - s);
+    unitDef = translateUnit(context->units, unit + s, len - s);
 
     if (unitDef == NULL) {
+        SCPI_ErrorPush(context, SCPI_ERROR_INVALID_SUFFIX);
         return FALSE;
     }
 
@@ -206,47 +250,51 @@
  */
 bool_t SCPI_ParamNumber(scpi_t * context, scpi_number_t * value, bool_t mandatory) {
     bool_t result;
-    char * param;
+    const char * param;
     size_t len;
     size_t numlen;
 
+    /* read parameter and shift to the next one */
     result = SCPI_ParamString(context, &param, &len, mandatory);
 
+    /* value not initializes */
     if (!value) {
         return FALSE;
     }
 
+    value->type = SCPI_NUM_DEF;
+
+    /* if parameter was not found, return TRUE or FALSE according
+     * to fact that parameter was mandatory or not */
     if (!result) {
-        if (mandatory) {
-            return FALSE;
-        } else {
-            value->type = SCPI_NUM_DEF;
-            return TRUE;
-        }
+        return mandatory ? FALSE : TRUE;
     }
 
-    value->unit = SCPI_UNIT_NONE;
-    value->value = 0.0;
-    value->type = translateSpecialNumber(context->special_numbers, param, len);
-
-    if (value->type != SCPI_NUM_NUMBER) {
-        // found special type
+    /* convert string to special number type */
+    if (translateSpecialNumber(context->special_numbers, param, len, value)) {
+        /* found special type */
         return TRUE;
     }
 
+    /* convert text from double - no special type */
     numlen = strToDouble(param, &value->value);
 
+    /* transform units of value */
     if (numlen <= len) {
-        if (transformNumber(context->units, param + numlen, len - numlen, value)) {
-            return TRUE;
-        } else {
-            SCPI_ErrorPush(context, SCPI_ERROR_INVALID_SUFFIX);
-        }
+        return transformNumber(context, param + numlen, len - numlen, value);
     }
     return FALSE;
 
 }
 
+/**
+ * Convert scpi_number_t to string
+ * @param context
+ * @param value number value
+ * @param str target string
+ * @param len max length of string
+ * @return number of chars written to string
+ */
 size_t SCPI_NumberToStr(scpi_t * context, scpi_number_t * value, char * str, size_t len) {
     const char * type;
     const char * unit;

--
Gitblit v1.9.1