From 926f29ed81a34a50c9221a2d562119db84a136cb Mon Sep 17 00:00:00 2001 From: David Robillard Date: Wed, 11 May 2011 22:27:18 +0000 Subject: Create SerdNodeFlags typedef for flags. Expose serd_strlen. Make serd_node_copy tolerate a NULL parameter. git-svn-id: http://svn.drobilla.net/serd/trunk@168 490d8e77-9747-427b-9fa3-0b8f29cee8a0 --- src/node.c | 38 +++++++++++++++++++++++++++++++++----- src/reader.c | 22 +++++++++++----------- src/serd_internal.h | 32 -------------------------------- 3 files changed, 44 insertions(+), 48 deletions(-) (limited to 'src') diff --git a/src/node.c b/src/node.c index edd0db86..e4bb87cf 100644 --- a/src/node.c +++ b/src/node.c @@ -19,6 +19,32 @@ #include "serd_internal.h" +SERD_API +size_t +serd_strlen(const uint8_t* str, size_t* n_bytes, SerdNodeFlags* flags) +{ + size_t n_chars = 0; + size_t i = 0; + for (; str[i]; ++i) { + if ((str[i] & 0xC0) != 0x80) { + // Does not start with `10', start of a new character + ++n_chars; + switch (str[i]) { + case '\r': + case '\n': + *flags |= SERD_HAS_NEWLINE; + break; + case '"': + *flags |= SERD_HAS_QUOTE; + } + } + } + if (n_bytes) { + *n_bytes = i + 1; + } + return n_chars; +} + SERD_API SerdNode serd_node_from_string(SerdType type, const uint8_t* buf) @@ -34,12 +60,14 @@ SERD_API SerdNode serd_node_copy(const SerdNode* node) { - SerdNode copy = *node; - if (node->buf) { - uint8_t* buf = malloc(copy.n_bytes); - memcpy(buf, node->buf, copy.n_bytes); - copy.buf = buf; + if (!node || !node->buf) { + return SERD_NODE_NULL; } + + SerdNode copy = *node; + uint8_t* buf = malloc(copy.n_bytes); + memcpy(buf, node->buf, copy.n_bytes); + copy.buf = buf; return copy; } diff --git a/src/reader.c b/src/reader.c index b0ad02e5..e956c3df 100644 --- a/src/reader.c +++ b/src/reader.c @@ -390,7 +390,7 @@ read_character_escape(SerdReader* reader, Ref dest) } static inline bool -read_echaracter_escape(SerdReader* reader, Ref dest, uint32_t* flags) +read_echaracter_escape(SerdReader* reader, Ref dest, SerdNodeFlags* flags) { switch (peek_byte(reader)) { case 't': @@ -413,7 +413,7 @@ read_echaracter_escape(SerdReader* reader, Ref dest, uint32_t* flags) } static inline bool -read_scharacter_escape(SerdReader* reader, Ref dest, uint32_t* flags) +read_scharacter_escape(SerdReader* reader, Ref dest, SerdNodeFlags* flags) { switch (peek_byte(reader)) { case '"': @@ -428,7 +428,7 @@ read_scharacter_escape(SerdReader* reader, Ref dest, uint32_t* flags) static inline bool read_ucharacter_escape(SerdReader* reader, Ref dest) { - uint32_t flags = 0; + SerdNodeFlags flags = 0; switch (peek_byte(reader)) { case '>': push_byte(reader, dest, eat_byte(reader, '>')); @@ -482,8 +482,8 @@ read_character(SerdReader* reader, Ref dest) static inline SerdStatus read_echaracter(SerdReader* reader, Ref dest) { - uint32_t flags = 0; - uint8_t c = peek_byte(reader); + SerdNodeFlags flags = 0; + uint8_t c = peek_byte(reader); switch (c) { case '\\': eat_byte(reader, '\\'); @@ -500,7 +500,7 @@ read_echaracter(SerdReader* reader, Ref dest) // [43] lcharacter ::= echaracter | '\"' | #x9 | #xA | #xD static inline SerdStatus -read_lcharacter(SerdReader* reader, Ref dest, uint32_t* flags) +read_lcharacter(SerdReader* reader, Ref dest, SerdNodeFlags* flags) { const uint8_t c = peek_byte(reader); uint8_t pre[3]; @@ -537,7 +537,7 @@ read_lcharacter(SerdReader* reader, Ref dest, uint32_t* flags) // [42] scharacter ::= ( echaracter - #x22 ) | '\"' static inline SerdStatus -read_scharacter(SerdReader* reader, Ref dest, uint32_t* flags) +read_scharacter(SerdReader* reader, Ref dest, SerdNodeFlags* flags) { uint8_t c = peek_byte(reader); switch (c) { @@ -621,7 +621,7 @@ read_ws_plus(SerdReader* reader) // [37] longString ::= #x22 #x22 #x22 lcharacter* #x22 #x22 #x22 static Ref -read_longString(SerdReader* reader, uint32_t* flags) +read_longString(SerdReader* reader, SerdNodeFlags* flags) { eat_string(reader, "\"\"\"", 3); Ref str = push_string(reader, "", 1); @@ -636,7 +636,7 @@ read_longString(SerdReader* reader, uint32_t* flags) // [36] string ::= #x22 scharacter* #x22 static Ref -read_string(SerdReader* reader, uint32_t* flags) +read_string(SerdReader* reader, SerdNodeFlags* flags) { eat_byte(reader, '\"'); Ref str = push_string(reader, "", 1); @@ -652,7 +652,7 @@ read_string(SerdReader* reader, uint32_t* flags) // [35] quotedString ::= string | longString static Ref -read_quotedString(SerdReader* reader, uint32_t* flags) +read_quotedString(SerdReader* reader, SerdNodeFlags* flags) { uint8_t pre[3]; peek_string(reader, pre, 3); @@ -903,7 +903,7 @@ read_resource(SerdReader* reader, Node* dest) // | integer | double | decimal | boolean static bool read_literal(SerdReader* reader, Node* dest, - Node* datatype, Ref* lang, uint32_t* flags) + Node* datatype, Ref* lang, SerdNodeFlags* flags) { Ref str = 0; const uint8_t c = peek_byte(reader); diff --git a/src/serd_internal.h b/src/serd_internal.h index e573a806..6c00fea1 100644 --- a/src/serd_internal.h +++ b/src/serd_internal.h @@ -98,36 +98,4 @@ is_digit(const uint8_t c) return in_range(c, '0', '9'); } -/** - UTF-8 strlen. - @return Length of @c str in characters. - @param str A null-terminated UTF-8 string. - @param n_bytes (Output) Set to the size of @a str in bytes (incl. NULL). -*/ -static inline size_t -serd_strlen(const uint8_t* str, size_t* n_bytes, uint32_t* flags) -{ - size_t n_chars = 0; - size_t i = 0; - for (; str[i]; ++i) { - if ((str[i] & 0xC0) != 0x80) { - // Does not start with `10', start of a new character - ++n_chars; - switch (str[i]) { - case '\r': - case '\n': - *flags |= SERD_HAS_NEWLINE; - break; - case '"': - *flags |= SERD_HAS_QUOTE; - } - } - } - if (n_bytes) { - *n_bytes = i + 1; - } - return n_chars; -} - - #endif // SERD_INTERNAL_H -- cgit v1.2.1