From e47406b803e010df9f10b723508335266c8421a2 Mon Sep 17 00:00:00 2001
From: ehofman <ehofman>
Date: Thu, 7 May 2009 12:23:30 +0000
Subject: [PATCH] more small fixes

---
 utils/xmlgrep/sample.xml |  2 +-
 utils/xmlgrep/testxml.c  | 32 ++++++++++++++++++++++++++++++++
 utils/xmlgrep/xml.c      | 13 ++++++++-----
 3 files changed, 41 insertions(+), 6 deletions(-)

diff --git a/utils/xmlgrep/sample.xml b/utils/xmlgrep/sample.xml
index 851089020..7fec2d03e 100644
--- a/utils/xmlgrep/sample.xml
+++ b/utils/xmlgrep/sample.xml
@@ -13,7 +13,7 @@
   <test n="2"/>
 
   <menu><name>* Traffic, # taxiing to runway (.</name></menu>
-  <menu><name>* Traffic, # holding short runway (.</name></menu>
+  <sample><test>  	* Traffic, # taxiing to runway (.    </test></sample>
 
   <!--
     - x is positive to the right
diff --git a/utils/xmlgrep/testxml.c b/utils/xmlgrep/testxml.c
index 318002ba9..dd22d457e 100644
--- a/utils/xmlgrep/testxml.c
+++ b/utils/xmlgrep/testxml.c
@@ -110,6 +110,38 @@ int main()
 
          free(node_id);
          free(path_id);
+
+         path_id = xmlNodeGet(root_id, "Configuration/output/sample/test");
+         if (path_id)
+         {
+            xmlNodeCopyString(root_id ,"Configuration/output/menu/name", buf, BUFLEN);
+            printf("Testing xmlCompareString against a fixed string: \t\t\t");
+            if (xmlCompareString(path_id, buf)) 	/* no match */
+               printf("failed.\n\t'%s' differs\n", buf);
+            else
+               printf("succes.\n");
+
+            s = xmlGetString(path_id);
+            if (s)
+            {
+               printf("Testing xmlGetString  against a fixed string: \t\t\t\t");
+               if (strcmp(s, buf))			/* mismatch */
+                  printf("failed.\n\t'%s' differs from\n\t'%s'\n", s, buf);
+               else
+                  printf("succes.\n");
+
+               printf("Testing xmlCopyString gainst a fixed string: \t\t\t\t");
+               xmlCopyString(path_id, buf, BUFLEN);
+               if (strcmp(s, buf))      		/* mismatch */
+                  printf("failed.\n\t'%s' differs from\n\t'%s'\n", s, buf);
+               else
+                  printf("succes.\n"); 
+
+               free(s);
+            }
+
+            free(path_id);
+         }
       }
       else
       {
diff --git a/utils/xmlgrep/xml.c b/utils/xmlgrep/xml.c
index f319dbfc2..9ee3ff6ac 100644
--- a/utils/xmlgrep/xml.c
+++ b/utils/xmlgrep/xml.c
@@ -465,11 +465,12 @@ xmlGetString(const void *id)
         char *ps;
 
         ps = xid->start;
-        len = xid->len;
+        len = xid->len-1;
         __xmlPrepareData(&ps, &len);
 
         if (len)
         {
+            len++;
             str = malloc(len+1);
             if (str)
             {
@@ -503,10 +504,11 @@ xmlCopyString(const void *id, char *buffer, size_t buflen)
         char *p;
 
         p = xid->start;
-        len = xid->len;
+        len = xid->len-1;
         __xmlPrepareData(&p, &len);
         if (len)
         {
+            len++;
             if (len >= buflen)
             {
                 len = buflen-1;
@@ -538,7 +540,7 @@ xmlCompareString(const void *id, const char *s)
         ps = xid->start;
         len = xid->len-1;
         __xmlPrepareData(&ps, &len);
-        ret = strncasecmp(ps, s, len);
+        ret = strncasecmp(ps, s, len+1);
     }
 
     return ret;
@@ -1638,7 +1640,7 @@ __xmlProcessCDATA(char **start, size_t *len)
             {
                 if ((restlen > 3) && (memcmp(new, "]]>", 3) == 0))
                 {
-                    *len = new-*start;
+                    *len = new - *start;
                     restlen -= 3;
                     new += 3;
                     break;
@@ -1717,7 +1719,8 @@ __xmlInfoProcess(const char *start, size_t len)
 }
 
 
-static void __xmlPrepareData(char **start, size_t *blocklen)
+static void
+__xmlPrepareData(char **start, size_t *blocklen)
 {
     size_t len = *blocklen;
     char *pe, *ps = *start;
-- 
2.39.5