From 7dca581454fb426dd8233a39475877dab9b3e1a6 Mon Sep 17 00:00:00 2001
From: Jan Breuer <jan.breuer@jaybee.cz>
Date: 周日, 04 10月 2015 19:57:28 +0800
Subject: [PATCH] Add Channel list support to readme

---
 libscpi/src/utils.c |   61 ++++++++++++++++++++----------
 1 files changed, 41 insertions(+), 20 deletions(-)

diff --git a/libscpi/src/utils.c b/libscpi/src/utils.c
index 2569570..e07d0f3 100644
--- a/libscpi/src/utils.c
+++ b/libscpi/src/utils.c
@@ -156,6 +156,19 @@
 }
 
 /**
+ * Converts string to unsigned 32bit integer representation
+ * @param str   string value
+ * @param val   32bit integer result
+ * @return      number of bytes used in string
+ */
+size_t strToULong(const char * str, uint32_t * val, int8_t base) {
+    char * endptr;
+    *val = strtoul(str, &endptr, base);
+    return endptr - str;
+}
+
+
+/**
  * Converts string to double representation
  * @param str   string value
  * @param val   double result
@@ -254,7 +267,7 @@
  * @param len - max search length
  * @return position of separator or len
  */
-size_t patternSeparatorShortPos(const char * pattern, size_t len) {
+static size_t patternSeparatorShortPos(const char * pattern, size_t len) {
     size_t i;
     for (i = 0; (i < len) && pattern[i]; i++) {
         if (islower((unsigned char) pattern[i])) {
@@ -270,7 +283,7 @@
  * @param len - max search length
  * @return position of separator or len
  */
-size_t patternSeparatorPos(const char * pattern, size_t len) {
+static size_t patternSeparatorPos(const char * pattern, size_t len) {
 
     char * separator = strnpbrk(pattern, len, "?:[]");
     if (separator == NULL) {
@@ -286,7 +299,7 @@
  * @param len - max search length
  * @return position of separator or len
  */
-size_t cmdSeparatorPos(const char * cmd, size_t len) {
+static size_t cmdSeparatorPos(const char * cmd, size_t len) {
     char * separator = strnpbrk(cmd, len, ":?");
     size_t result;
     if (separator == NULL) {
@@ -309,7 +322,7 @@
 scpi_bool_t matchPattern(const char * pattern, size_t pattern_len, const char * str, size_t str_len, int32_t * num) {
     int pattern_sep_pos_short;
 
-    if (pattern[pattern_len - 1] == '#') {
+    if ((pattern_len > 0) && pattern[pattern_len - 1] == '#') {
         size_t new_pattern_len = pattern_len - 1;
 
         pattern_sep_pos_short = patternSeparatorShortPos(pattern, new_pattern_len);
@@ -338,7 +351,7 @@
     int rightFlag = 0; // flag for ']' on right
     int cmd_sep_pos = 0;
 
-    size_t numbers_idx = -1;
+    size_t numbers_idx = 0;
     int32_t *number_ptr = NULL;
 
     const char * pattern_ptr = pattern;
@@ -378,20 +391,21 @@
             cmd_sep_pos = cmdSeparatorPos(cmd_ptr, cmd_end - cmd_ptr);
         }
 
-        if (pattern_ptr[pattern_sep_pos - 1] == '#') {
-            numbers_idx++;
+        if ((pattern_sep_pos > 0) && pattern_ptr[pattern_sep_pos - 1] == '#') {
             if (numbers && (numbers_idx < numbers_len)) {
                 number_ptr = numbers + numbers_idx;
                 *number_ptr = 1; // default value
             } else {
                 number_ptr = NULL;
             }
+            numbers_idx++;
         } else {
             number_ptr = NULL;
         }
 
         if (matchPattern(pattern_ptr, pattern_sep_pos, cmd_ptr, cmd_sep_pos, number_ptr)) {
-            pattern_ptr = pattern_ptr + pattern_sep_pos;
+            pattern_ptr += pattern_sep_pos;
+            pattern_len -= pattern_sep_pos;
             cmd_ptr = cmd_ptr + cmd_sep_pos;
             result = TRUE;
 
@@ -422,24 +436,28 @@
 
             /* both command and patter contains command separator at this position */
             if ((pattern_ptr[0] == cmd_ptr[0]) && ((pattern_ptr[0] == ':') || (pattern_ptr[0] == '?'))) {
-                pattern_ptr = pattern_ptr + 1;
+                pattern_ptr += 1;
+                pattern_len -= 1;
                 cmd_ptr = cmd_ptr + 1;
-            } else if ((pattern_ptr[1] == cmd_ptr[0])
+            } else if ((pattern_len > 1) && (pattern_ptr[1] == cmd_ptr[0])
                     && (pattern_ptr[0] == '[')
                     && (pattern_ptr[1] == ':')) {
-                pattern_ptr = pattern_ptr + 2; // for skip '[' in "[:"
+                pattern_ptr += 2; // for skip '[' in "[:"
+                pattern_len -= 2;
                 cmd_ptr = cmd_ptr + 1;
                 leftFlag++;
-            } else if ((pattern_ptr[1] == cmd_ptr[0])
+            } else if ((pattern_len > 1) &&  (pattern_ptr[1] == cmd_ptr[0])
                     && (pattern_ptr[0] == ']')
                     && (pattern_ptr[1] == ':')) {
-                pattern_ptr = pattern_ptr + 2; // for skip ']' in "]:"
+                pattern_ptr += 2; // for skip ']' in "]:"
+                pattern_len -= 2;
                 cmd_ptr = cmd_ptr + 1;
-            } else if ((pattern_ptr[2] == cmd_ptr[0])
+            } else if ((pattern_len > 2) && (pattern_ptr[2] == cmd_ptr[0])
                     && (pattern_ptr[0] == ']')
                     && (pattern_ptr[1] == '[')
                     && (pattern_ptr[2] == ':')) {
-                pattern_ptr = pattern_ptr + 3; // for skip '][' in "][:"
+                pattern_ptr += 3; // for skip '][' in "][:"
+                pattern_len -= 3;
                 cmd_ptr = cmd_ptr + 1;
                 leftFlag++;
             } else if (((pattern_ptr[0] == ']')
@@ -453,14 +471,17 @@
                 break;
             }
         } else {
-            pattern_ptr = pattern_ptr + pattern_sep_pos;
+            pattern_ptr += pattern_sep_pos;
+            pattern_len -= pattern_sep_pos;
             if ((pattern_ptr[0] == ']') && (pattern_ptr[1] == ':')) {
-                pattern_ptr = pattern_ptr + 2; // for skip ']' in "]:" , pattern_ptr continue, while cmd_ptr remain unchanged
+                pattern_ptr += 2; // for skip ']' in "]:" , pattern_ptr continue, while cmd_ptr remain unchanged
+                pattern_len -= 2;
                 rightFlag++;
-            } else if ((pattern_ptr[0] == ']')
+            } else if ((pattern_len > 2) && (pattern_ptr[0] == ']')
                     && (pattern_ptr[1] == '[')
                     && (pattern_ptr[2] == ':')) {
-                pattern_ptr = pattern_ptr + 3; // for skip ']' in "][:" , pattern_ptr continue, while cmd_ptr remain unchanged
+                pattern_ptr += 3; // for skip ']' in "][:" , pattern_ptr continue, while cmd_ptr remain unchanged
+                pattern_len -= 3;
                 rightFlag++;
             } else {
                 result = FALSE;
@@ -548,7 +569,7 @@
         if (c1 != c2) {
             return c1 - c2;
         }
-        if (c1 = '\0') {
+        if (c1 == '\0') {
             return 0;
         }
     }

--
Gitblit v1.9.1