From: Linus Torvalds <torvalds@linux-foundation.org>
Date: Tue, 30 Aug 2011 04:32:27 +0000 (-0700)
Subject: Move the parser closer to being usable
X-Git-Url: http://git.tdb.fi/?a=commitdiff_plain;ds=sidebyside;h=a71e8dd777a27b5e9c56e1834f964b385b1157d4;p=ext%2Fsubsurface.git

Move the parser closer to being usable

Ok, so we have the dive split and the sample splits, so now we could
really just start filling in data.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/parse.c b/parse.c
index ffb01d2..2321f7d 100644
--- a/parse.c
+++ b/parse.c
@@ -4,6 +4,35 @@
 #include <libxml/parser.h>
 #include <libxml/tree.h>
 
+/*
+ * File boundaries are dive boundaries. But sometimes there are
+ * multiple dives per file, so there can be other events too that
+ * trigger a "new dive" marker and you may get some nesting due
+ * to that. Just ignore nesting levels.
+ */
+static void dive_start(void)
+{
+	printf("---\n");
+}
+
+static void dive_end(void)
+{
+}
+
+static void sample_start(void)
+{
+	printf("Sample:\n");
+}
+
+static void sample_end(void)
+{
+}
+
+static void entry(const char *name, int size, const char *buffer)
+{
+	printf("%s: %.*s\n", name, size, buffer);
+}
+
 static const char *nodename(xmlNode *node, char *buf, int len)
 {
 	/* Don't print out the node name if it is "text" */
@@ -37,7 +66,7 @@ static const char *nodename(xmlNode *node, char *buf, int len)
 
 #define MAXNAME 64
 
-static void show_one_node(xmlNode *node)
+static void visit_one_node(xmlNode *node)
 {
 	int len;
 	const unsigned char *content;
@@ -62,16 +91,38 @@ static void show_one_node(xmlNode *node)
 
 	name = nodename(node, buffer, sizeof(buffer));
 
-	printf("%s: %.*s\n", name, len, content);
+	entry(name, len, content);
 }
 
-static void show(xmlNode *node)
+static void traverse(xmlNode *node)
 {
 	xmlNode *n;
 
 	for (n = node; n; n = n->next) {
-		show_one_node(n);
-		show(n->children);
+		/* XML from libdivecomputer: 'dive' per new dive */
+		if (!strcmp(n->name, "dive")) {
+			dive_start();
+			traverse(n->children);
+			dive_end();
+			continue;
+		}
+
+		/*
+		 * At least both libdivecomputer and Suunto
+		 * agree on "sample".
+		 *
+		 * Well - almost. Ignore case.
+		 */
+		if (!strcasecmp(n->name, "sample")) {
+			sample_start();
+			traverse(n->children);
+			sample_end();
+			continue;
+		}
+
+		/* Anything else - just visit it and recurse */
+		visit_one_node(n);
+		traverse(n->children);
 	}
 }
 
@@ -85,7 +136,9 @@ static void parse(const char *filename)
 		return;
 	}
 
-	show(xmlDocGetRootElement(doc));
+	dive_start();
+	traverse(xmlDocGetRootElement(doc));
+	dive_end();
 	xmlFreeDoc(doc);
 	xmlCleanupParser();
 }