Token manipulation functions changes. size_t replaced with unsigned int.

demo.c

@@ -95,26 +95,26 @@ int main(int argc, char *argv[]) {
 		exit(EXIT_FAILURE);
 	}
 
+	jsmn_parser parser;
+	jsmn_init_parser(&parser, js, tokens, num_tokens);
+
 	while (1) {
 		char buf[BUFSIZ];
-		r = fread(buf, 1, BUFSIZ, f);
+		r = fread(buf, 1, 1, f);
 		if (r <= 0) {
 			break;
 		}
-		js = (char *) realloc(js, filesize + r);
+		js = (char *) realloc(js, filesize + r + 1);
 		if (js == NULL) {
 			fprintf(stderr, "Cannot allocate anough memory\n");
 			fclose(f);
 			exit(EXIT_FAILURE);
 		}
+		parser.js = js;
+
 		memcpy(js + filesize, buf, r);
 		filesize += r;
-	}
+		js[filesize] = '\0';
-
-	fclose(f);
-
-	jsmn_parser parser;
-	jsmn_init_parser(&parser, js, tokens, num_tokens);
 
 		r = jsmn_parse(&parser);
 		if (r < 0) {
@@ -124,7 +124,10 @@ int main(int argc, char *argv[]) {
 		for (i = 0; i<num_tokens; i++) {
 			jsmn_dump_obj(&parser.tokens[i], js);
 		}
+	}
 
+	fclose(f);
+	free(tokens);
 	free(js);
 
 	return 0;

jsmn.c

@@ -2,67 +2,63 @@
 
 #include "jsmn.h"
 
-void jsmn_init_parser(jsmn_parser *parser, const char *js, 
+static jsontok_t *jsmn_get_token(jsmn_parser *parser) {
-                      jsontok_t *tokens, size_t num_tokens) {
-	unsigned int i;
-
-	parser->js = js;
-	parser->pos = 0;
-	parser->tokens = tokens;
-	parser->num_tokens = num_tokens;
-
-	for (i = 0; i < parser->num_tokens; i++) {
-		parser->tokens[i].start = -1;
-		parser->tokens[i].end = -1;
-		parser->tokens[i].type = JSON_PRIMITIVE;
-	}
-}
-
-jsontok_t *jsmn_start_token(jsmn_parser *parser, jsontype_t type) {
 	unsigned int i;
 	jsontok_t *tokens = parser->tokens;
-	for (i = 0; i<parser->num_tokens; i++) {
+	for (i = parser->curtoken; i<parser->num_tokens; i++) {
 		if (tokens[i].start == -1 && tokens[i].end == -1) {
-			tokens[i].start = parser->pos;
+			parser->curtoken = i;
-			tokens[i].type = type;
 			return &tokens[i];
 		}
 	}
 	return NULL;
 }
 
-jsontok_t *jsmn_end_token(jsmn_parser *parser, jsontype_t type) {
+static void jsmn_fill_token(jsontok_t *token, jsontype_t type, int start, int end) {
-	int i;
+	token->type = type;
-	jsontok_t *tokens = parser->tokens;
+	token->start = start;
-	for (i = parser->num_tokens - 1; i>= 0; i--) {
+	token->end = end;
-		if (tokens[i].type == type && tokens[i].start != -1 && tokens[i].end == -1) {
+}
-			tokens[i].end = parser->pos;
+
-			return &tokens[i];
+void jsmn_init_parser(jsmn_parser *parser, const char *js, 
-		}
+                      jsontok_t *tokens, unsigned int num_tokens) {
+	unsigned int i;
+
+	parser->js = js;
+	parser->pos = 0;
+	parser->tokens = tokens;
+	parser->num_tokens = num_tokens;
+	parser->curtoken = 0;
+
+	for (i = 0; i < parser->num_tokens; i++) {
+		jsmn_fill_token(&parser->tokens[i], JSON_PRIMITIVE, -1, -1);
 	}
-	return NULL;
 }
 
 static int jsmn_parse_primitive(jsmn_parser *parser) {
 	const char *js;
 	jsontok_t *token;
+	int start;
 
-	js = parser->js;
+	start = parser->pos;
 
-	token = jsmn_start_token(parser, JSON_PRIMITIVE);
+	js = parser->js;
 
 	for (; js[parser->pos] != '\0'; parser->pos++) {
 		switch (js[parser->pos]) {
 			case '\t' : case '\r' : case '\n' : case ' ' :
 			case ','  : case ']'  : case '}' :
-				token->end = parser->pos;
+				token = jsmn_get_token(parser);
+				jsmn_fill_token(token, JSON_PRIMITIVE, start, parser->pos);
 				parser->pos--;
 				return JSMN_SUCCESS;
 		}
 		if (js[parser->pos] < 32 || js[parser->pos] >= 127) {
+			parser->pos = start;
 			return JSMN_ERROR_INVAL;
 		}
 	}
+	parser->pos = start;
 	return JSMN_ERROR_PART;
 }
 
@@ -70,23 +66,20 @@ static int jsmn_parse_string(jsmn_parser *parser) {
 	const char *js;
 	jsontok_t *token;
 
-	js = parser->js;
+	int start = parser->pos;
 
-	/* Check if string begins from a quote */
+	js = parser->js;
-	if (js[parser->pos] != '\"') {
-		return JSMN_ERROR_INVAL;
-	}
 
 	parser->pos++;
 
-	token = jsmn_start_token(parser, JSON_STRING);
 	/* Skip starting quote */
 	for (; js[parser->pos] != '\0'; parser->pos++) {
 		char c = js[parser->pos];
 
 		/* Quote: end of string */
 		if (c == '\"') {
-			token->end = parser->pos;
+			token = jsmn_get_token(parser);
+			jsmn_fill_token(token, JSON_PRIMITIVE, start, parser->pos);
 			return JSMN_SUCCESS;
 		}
 
@@ -104,14 +97,18 @@ static int jsmn_parse_string(jsmn_parser *parser) {
 					break;
 				/* Unexpected symbol */
 				default:
+					parser->pos = start;
 					return JSMN_ERROR_INVAL;
 			}
 		}
 	}
+	parser->pos = start;
 	return JSMN_ERROR_PART;
 }
 
 jsmnerr_t jsmn_parse(jsmn_parser *parser) {
+	int r;
+	unsigned int i;
 	const char *js;
 	jsontype_t type;
 	jsontok_t *token;
@@ -123,21 +120,32 @@ jsmnerr_t jsmn_parse(jsmn_parser *parser) {
 		c = js[parser->pos];
 		switch (c) {
 			case '{': case '[':
-				type = (c == '{' ? JSON_OBJECT : JSON_ARRAY);
+				token = jsmn_get_token(parser);
-				token = jsmn_start_token(parser, type);
+				token->type = (c == '{' ? JSON_OBJECT : JSON_ARRAY);
+				token->start = parser->pos;
 				break;
 			case '}': case ']':
 				type = (c == '}' ? JSON_OBJECT : JSON_ARRAY);
-				token = jsmn_end_token(parser, type);
+				for (i = parser->curtoken; i >= 0; i--) {
-				token->end++;
+					token = &parser->tokens[i];
+					if (token->start != -1 && token->end == -1) {
+						if (token->type != type) {
+							return JSMN_ERROR_INVAL;
+						}
+						token->end = parser->pos + 1;
+						break;
+					}
+				}
 				break;
 			case '-': case '0': case '1' : case '2': case '3' : case '4':
 			case '5': case '6': case '7' : case '8': case '9':
 			case 't': case 'f': case 'n' :
-				jsmn_parse_primitive(parser);
+				r = jsmn_parse_primitive(parser);
+				if (r < 0) return r;
 				break;
 			case '\"':
-				jsmn_parse_string(parser);
+				r = jsmn_parse_string(parser);
+				if (r < 0) return r;
 				break;
 			case '\t' : case '\r' : case '\n' : case ':' : case ',': case ' ': 
 				break;

jsmn.h

@@ -6,8 +6,7 @@
  * 	o Object
  * 	o Array
  * 	o String
- * 	o Number
+ * 	o Other primitive: number, boolean (true/false) or null
- * 	o Other primitive: boolean (true/false) or null
  */
 typedef enum {
 	JSON_PRIMITIVE = 0,
@@ -42,7 +41,8 @@ typedef struct {
 typedef struct {
 	const char *js;
 	unsigned int pos;
-	size_t num_tokens;
+	unsigned int num_tokens;
+	int curtoken;
 	jsontok_t *tokens;
 } jsmn_parser;
 
@@ -50,7 +50,7 @@ typedef struct {
  * Create JSON parser over an array of tokens
  */
 void jsmn_init_parser(jsmn_parser *parser, const char *js, 
-                      jsontok_t *tokens, size_t num_tokens);
+                      jsontok_t *tokens, unsigned int num_tokens);
 
 /**
  * Run JSON parser. It parses a JSON data string into and array of tokens, each describing