From 0993d312cb18131de8aa5a73eda65305f8f7899b Mon Sep 17 00:00:00 2001
From: David Robillard <d@drobilla.net>
Date: Sun, 23 Jan 2011 20:18:20 +0000
Subject: Faster serd_uri_serialise.

git-svn-id: http://svn.drobilla.net/serd/trunk@50 490d8e77-9747-427b-9fa3-0b8f29cee8a0
---
 src/uri.c | 51 ++++++++++++++++++++++++++++++++-------------------
 1 file changed, 32 insertions(+), 19 deletions(-)

(limited to 'src/uri.c')

diff --git a/src/uri.c b/src/uri.c
index 5b2e4423..6fb6d42e 100644
--- a/src/uri.c
+++ b/src/uri.c
@@ -269,27 +269,40 @@ serd_uri_serialise(const SerdURI* uri, SerdSink sink, void* stream)
 			/* Merge paths, removing dot components.
 			   See http://tools.ietf.org/html/rfc3986#section-5.2.3
 			*/
-			const uint8_t* uri_first = uri->path.buf;
-			const uint8_t* uri_end   = uri_first;
+			const uint8_t* begin = uri->path.buf;
+			const uint8_t* end   = begin;
 			size_t         up        = 1;
-			if (uri_first) {
+			if (begin) {
 				// Count and skip leading dot components
-				uri_end = uri->path.buf + uri->path.len;
-				while (uri_first < uri_end) {
-					if (!memcmp((const char*)uri_first, "./", 2)) {
-						uri_first += 2;
-					} else if (!memcmp((const char*)uri_first, "../", 3)) {
-						++up;
-						uri_first += 3;
-					} else if (!memcmp((const char*)uri_first, "..", 2)) {
-						++up;
-						uri_first += 2;
-					} else if (!memcmp((const char*)uri_first, ".", 1)) {
-						++uri_first;
-					} else if (!memcmp((const char*)uri_first, "//", 1)) {
-						++uri_first;
-					} else {
+				end = uri->path.buf + uri->path.len;
+				for (bool done = false; !done && (begin < end);) {
+					switch (begin[0]) {
+					case '.':
+						switch (begin[1]) {
+						case '/':
+							begin += 2;  // Chop leading "./"
+							break;
+						case '.':
+							++up;
+							switch (begin[2]) {
+							case '/':
+								begin += 3;  // Chop lading "../"
+								break;
+							default:
+								begin += 2;  // Chop leading ".."
+							}
+							break;
+						default:
+							++begin;  // Chop leading "."
+						}
 						break;
+					case '/':
+						if (begin[1] == '/') {
+							++begin;  // Replace leading "//" with "/"
+							break;
+						}  // else fall through
+					default:
+						done = true;  // Finished chopping dot components
 					}
 				}
 
@@ -312,7 +325,7 @@ serd_uri_serialise(const SerdURI* uri, SerdSink sink, void* stream)
 				}
 
 				// Write URI suffix
-				WRITE(uri_first, uri_end - uri_first);
+				WRITE(begin, end - begin);
 			}
 		}
 	} else {
-- 
cgit v1.2.1